New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[GIE] impl graph_proxy for csr_store #2466

Merged

lnfjpt merged 54 commits into alibaba:main from lnfjpt:main

Mar 21, 2023

Collaborator

lnfjpt commented Feb 27, 2023 •

edited by sighingnow

Related issue number

Fixes #2465

lnfjpt requested review from luoxiaojian and BingqingLyu

February 27, 2023 05:14

codecov-commenter commented Feb 27, 2023 •

edited

Codecov Report

Merging #2466 (f396210) into main (8b4103f) will increase coverage by 32.69%.
The diff coverage is 53.84%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2466       +/-   ##
===========================================
+ Coverage   39.88%   72.57%   +32.69%     
===========================================
  Files          88       88               
  Lines        9814     9838       +24     
===========================================
+ Hits         3914     7140     +3226     
+ Misses       5900     2698     -3202

Impacted Files	Coverage Δ
python/graphscope/client/session.py	`75.88% <ø> (+10.09%)`	⬆️
...on/graphscope/tests/kubernetes/test_demo_script.py	`72.15% <50.00%> (-3.50%)`	⬇️
python/graphscope/config.py	`100.00% <100.00%> (ø)`
python/graphscope/deploy/kubernetes/cluster.py	`76.19% <100.00%> (ø)`

... and 46 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

lnfjpt force-pushed the main branch 2 times, most recently from eed91c5 to b3584ac Compare

March 9, 2023 02:03

lnfjpt changed the title ~~[WIP] impl graph_proxy for csr_store~~ [GIE] impl graph_proxy for csr_store

longbinlai reviewed

View reviewed changes

interactive_engine/executor/store/mcsr/src/graph_loader.rs Show resolved Hide resolved

longbinlai reviewed

View reviewed changes

interactive_engine/executor/store/mcsr/src/graph_partitioner.rs Show resolved Hide resolved

longbinlai reviewed

View reviewed changes

interactive_engine/executor/store/mcsr/src/graph_partitioner.rs Outdated Show resolved Hide resolved

longbinlai reviewed

View reviewed changes

interactive_engine/executor/store/mcsr/src/date.rs Show resolved Hide resolved

longbinlai reviewed

View reviewed changes

interactive_engine/executor/store/mcsr/src/date_time.rs Show resolved Hide resolved

longbinlai reviewed

View reviewed changes

Collaborator

longbinlai left a comment •

edited

Still many copyrights are missing.

longbinlai reviewed

View reviewed changes

Collaborator

longbinlai left a comment

Can include some unit tests.

lnfjpt force-pushed the main branch 6 times, most recently from b2a101c to a14fb60 Compare

March 20, 2023 08:54

lnfjpt added 11 commits

March 21, 2023 14:07


          impl graph_proxy for csr store

03ee814


          update schema

73bb550


          rm rsmpi

183f917


          support edge properties

cc472b1


          update local_edge & traverse

61d7405


          fix edge property in mcsr

162d672


          rm offset in adjlist


          add LazyEdgeDetails

6625a6a


          rebase

5ade962


          rm adj_degree in mcsr

79b9e7f


          avoid duplicate properties

969ed52

lnfjpt and others added 27 commits

March 21, 2023 14:07


          rm log when property not exist

1cb13fd


          add Copyright

9d00880


          update datetime implementation

eb81068


          update date implementation

ebcc3da


          rm unused interface

bb8cc36


          update

25382d9


          support date in read graph

e5345c5


          update raw data partitioner

0c63ab2


          rm unused implementation

b46db16


          tidy up

41d779e


          rm adhoc

1fd4ceb


          update col_table serial/deserial

cebe61f


          support scan edges in csr store

eb60b76


          support double on property


          update get_all_properties

299c630


          support skip head

5db2e30


          add readme for csr store

cd84a7a


          update query 11 in pattern ci

d389e20

fmt

72d0fe1


          fix rebase

9d58f9b


          add test graph for mcsr

d83d7ea


          update

74fd217


          [GAIA Compiler] minor fix and format java codes

1518be0


          add ci for csr store

507ac21


          fix k8s ci

20bcb44


          [GAIA Compiler] minor fix

92c36f2


          [GAIA Compiler] minor fix

39701d1

lnfjpt force-pushed the main branch from f396210 to 39701d1 Compare

March 21, 2023 06:08

luoxiaojian approved these changes

View reviewed changes

lnfjpt merged commit 8a215c3 into alibaba:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment