GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

impl new condition row filter and column filter

Open waruto210 opened this issue 3 years ago • 1 comments
trafficstars

What do these changes do?

impl new condition filter

  • impl Condition to filter vertex/edge
  • make GlobalGraph support row filter

make GlobalGraph support column filter

make GraphScopeStore support filter

harmonize the output_properties format of ir and groot

add a column_filter_pushdown feature to control column filter

Related issue number

#1530

waruto210 avatar Jul 31 '22 05:07 waruto210

Codecov Report

Merging #1905 (b621d92) into main (2ac4f24) will decrease coverage by 3.80%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1905      +/-   ##
==========================================
- Coverage   44.04%   40.23%   -3.81%     
==========================================
  Files          89       89              
  Lines        9834     9834              
==========================================
- Hits         4331     3957     -374     
- Misses       5503     5877     +374     
Impacted Files Coverage Δ
python/graphscope/tests/unittest/test_java_app.py 0.00% <0.00%> (-100.00%) :arrow_down:
python/graphscope/deploy/hosts/cluster.py 29.41% <0.00%> (-58.83%) :arrow_down:
python/graphscope/analytical/app/java_app.py 24.24% <0.00%> (-44.95%) :arrow_down:
python/graphscope/framework/utils.py 40.10% <0.00%> (-14.85%) :arrow_down:
python/graphscope/framework/app.py 69.95% <0.00%> (-10.33%) :arrow_down:
python/graphscope/client/rpc.py 81.48% <0.00%> (-8.34%) :arrow_down:
python/graphscope/framework/graph_schema.py 55.28% <0.00%> (-6.98%) :arrow_down:
python/graphscope/tests/conftest.py 48.21% <0.00%> (-4.65%) :arrow_down:
python/graphscope/client/session.py 63.75% <0.00%> (-3.22%) :arrow_down:
python/graphscope/analytical/udf/utils.py 36.17% <0.00%> (-3.20%) :arrow_down:
... and 3 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 2ac4f24...b621d92. Read the comment docs.

codecov-commenter avatar Aug 01 '22 18:08 codecov-commenter

Hi @waruto210,

The previous CI failure on GAIA test should have been addressed in https://github.com/alibaba/GraphScope/pull/1979, could you please rebase your branch against current main and see if CI comes green again?

Thanks!

sighingnow avatar Aug 22 '22 07:08 sighingnow