GraphScope
GraphScope copied to clipboard
[FEATURE-287]:Session persistence and restore #287
Signed-off-by: bifenglin [email protected]
What do these changes do?
Session
persistence and restore, It includes session config and graph object.
Related issue number
#287
Fixes
Codecov Report
Merging #1986 (43fccc5) into main (bd8252f) will decrease coverage by
33.25%
. The diff coverage is2.91%
.
@@ Coverage Diff @@
## main #1986 +/- ##
===========================================
- Coverage 73.10% 39.85% -33.26%
===========================================
Files 89 90 +1
Lines 9834 9937 +103
===========================================
- Hits 7189 3960 -3229
- Misses 2645 5977 +3332
Impacted Files | Coverage Δ | |
---|---|---|
python/graphscope/client/test.py | 0.00% <0.00%> (ø) |
|
python/graphscope/tests/unittest/test_session.py | 0.00% <0.00%> (-64.22%) |
:arrow_down: |
python/graphscope/client/session.py | 58.15% <5.08%> (-16.85%) |
:arrow_down: |
python/graphscope/tests/unittest/test_java_app.py | 0.00% <0.00%> (-100.00%) |
:arrow_down: |
...ython/graphscope/tests/unittest/test_cython_ast.py | 0.00% <0.00%> (-100.00%) |
:arrow_down: |
...ython/graphscope/tests/unittest/test_serailaize.py | 0.00% <0.00%> (-100.00%) |
:arrow_down: |
python/graphscope/analytical/udf/patch.py | 3.47% <0.00%> (-96.53%) |
:arrow_down: |
python/graphscope/tests/unittest/test_lazy.py | 0.00% <0.00%> (-96.30%) |
:arrow_down: |
...thon/graphscope/tests/unittest/test_scalability.py | 0.00% <0.00%> (-96.16%) |
:arrow_down: |
python/graphscope/tests/unittest/test_app.py | 0.00% <0.00%> (-95.66%) |
:arrow_down: |
... and 42 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 bd8252f...43fccc5. Read the comment docs.
So, If there is any request or any problem with this PR. Plz remind me. And I will work on it until it has been satisfied. @siyuan0322
So, If there is any request or any problem with this PR. Plz remind me. And I will work on it until it has been satisfied. @siyuan0322
Sorry for I'm thinking to drop that graph_list
parameters so that it will automatically save all opened graphs, but that seems a little hard to implement, and while I'm thinking about the solution, my attention was drawn to other issues = =. I'll pick it up and get it merged ASAP this time.
So, If there is any request or any problem with this PR. Plz remind me. And I will work on it until it has been satisfied. @siyuan0322
Sorry for I'm thinking to drop that
graph_list
parameters so that it will automatically save all opened graphs, but that seems a little hard to implement, and while I'm thinking about the solution, my attention was drawn to other issues = =. I'll pick it up and get it merged ASAP this time.
Thanks for your reply. And I have known the request. I will follow the request in the issue so that it can clear the requirement.