GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

[GAE-Java] GraphX-on-GraphScope : remove `GraphXFragment` and related components

Open zhanglei1949 opened this issue 3 years ago • 1 comments

What do these changes do?

Background

The original GraphX-on-GraphScope implementation support both GraphXFragment And ArrowProjectedFragment. Relying on two storage structure introducing unnecessary complexity and GraphXFragment doesn't differ much from ArrowProjectedFragment from a design point of view . So in this PR, we remove GraphXFragment and related components, and refactor Graphx-on-GraphScope to adapt this change.

zhanglei1949 avatar Oct 10 '22 12:10 zhanglei1949

Codecov Report

Merging #2123 (c22f099) into main (49ae814) will decrease coverage by 1.22%. The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2123      +/-   ##
==========================================
- Coverage   72.93%   71.71%   -1.23%     
==========================================
  Files          89       89              
  Lines        9789     9789              
==========================================
- Hits         7140     7020     -120     
- Misses       2649     2769     +120     
Impacted Files Coverage Δ
python/graphscope/tests/unittest/test_java_app.py 53.01% <0.00%> (-46.99%) :arrow_down:
python/graphscope/analytical/app/java_app.py 24.36% <0.00%> (-27.92%) :arrow_down:
python/graphscope/client/rpc.py 83.20% <0.00%> (-3.06%) :arrow_down:
python/graphscope/framework/graph_utils.py 80.15% <0.00%> (-2.39%) :arrow_down:
python/graphscope/client/session.py 74.95% <0.00%> (-1.52%) :arrow_down:
python/graphscope/framework/app.py 90.32% <0.00%> (-1.39%) :arrow_down:
python/graphscope/framework/utils.py 66.40% <0.00%> (-1.05%) :arrow_down:
python/graphscope/framework/graph_schema.py 64.61% <0.00%> (-0.23%) :arrow_down:

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 49ae814...c22f099. Read the comment docs.

codecov-commenter avatar Oct 12 '22 07:10 codecov-commenter