RedisGraph icon indicating copy to clipboard operation
RedisGraph copied to clipboard

Add copy functionality and clone routines

Open jeffreylovitz opened this issue 2 years ago • 4 comments

This PR adds support for the Redis COPY command.

jeffreylovitz avatar Mar 03 '22 18:03 jeffreylovitz

Codecov Report

Merging #2189 (a919d55) into master (38bbc25) will decrease coverage by 0.86%. The diff coverage is 20.71%.

:exclamation: Current head a919d55 differs from pull request most recent head ffaf323. Consider uploading reports for the commit ffaf323 to get more accurate results

@@            Coverage Diff             @@
##           master    #2189      +/-   ##
==========================================
- Coverage   93.08%   92.22%   -0.87%     
==========================================
  Files         253      251       -2     
  Lines       22657    22689      +32     
==========================================
- Hits        21090    20924     -166     
- Misses       1567     1765     +198     
Impacted Files Coverage Δ
src/graph/entities/graph_entity.c 48.27% <0.00%> (-2.82%) :arrow_down:
src/graph/entities/node.c 0.00% <ø> (ø)
src/graph/graph.c 96.27% <0.00%> (-3.03%) :arrow_down:
src/graph/graph_statistics.c 82.14% <0.00%> (-17.86%) :arrow_down:
src/graph/graph_statistics.h 100.00% <ø> (ø)
src/graph/graphcontext.c 89.67% <0.00%> (-7.83%) :arrow_down:
src/schema/schema.c 88.88% <0.00%> (-3.62%) :arrow_down:
src/util/datablock/datablock.c 79.38% <8.69%> (-16.23%) :arrow_down:
src/serializers/graphcontext_type.c 79.06% <16.66%> (-10.12%) :arrow_down:
src/index/index.c 88.62% <20.83%> (-7.77%) :arrow_down:
... and 100 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 38bbc25...ffaf323. Read the comment docs.

codecov[bot] avatar Mar 04 '22 19:03 codecov[bot]

@jeffreylovitz I've started refactoring some of the logic, can you please review my changes and address the TODO comments ?

swilly22 avatar Mar 07 '22 08:03 swilly22

#1076 #2366

swilly22 avatar May 28 '22 08:05 swilly22

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 22 '22 05:09 CLAassistant