Skip to content
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.

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

[GAIA-IR] Map KeyId back into KeyName for labels and properties #1615

Merged
merged 70 commits into from
May 26, 2022
Merged

[GAIA-IR] Map KeyId back into KeyName for labels and properties #1615

merged 70 commits into from
May 26, 2022

Conversation

BingqingLyu
Copy link
Collaborator

What do these changes do?

Map a label/property id back into label/property name in compiler, with id-name-mapping method provided in ir-core.

Related issue number

#1521

longbinlai and others added 30 commits May 26, 2021 14:41
…oid potential cyclice dependencies while using the library in pegasus.
[IR Runtime] IR TagOpt for more compact structure in Runtime
@BingqingLyu BingqingLyu removed the wip label May 24, 2022
@BingqingLyu BingqingLyu changed the title [IR] Map KeyId back into KeyName for labels and properties [GAIA-IR] Map KeyId back into KeyName for labels and properties May 24, 2022
@codecov-commenter
Copy link

codecov-commenter commented May 25, 2022

Codecov Report

Merging #1615 (3dbef24) into main (fa5718c) will decrease coverage by 32.84%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1615       +/-   ##
===========================================
- Coverage   73.15%   40.31%   -32.85%     
===========================================
  Files          89       89               
  Lines        9727     9727               
===========================================
- Hits         7116     3921     -3195     
- Misses       2611     5806     +3195     
Impacted Files Coverage Δ
python/graphscope/tests/unittest/test_java_app.py 0.00% <0.00%> (-100.00%) ⬇️
...ython/graphscope/tests/unittest/test_cython_ast.py 0.00% <0.00%> (-100.00%) ⬇️
...ython/graphscope/tests/unittest/test_serailaize.py 0.00% <0.00%> (-100.00%) ⬇️
python/graphscope/analytical/udf/patch.py 3.47% <0.00%> (-96.53%) ⬇️
python/graphscope/tests/unittest/test_lazy.py 0.00% <0.00%> (-96.30%) ⬇️
...thon/graphscope/tests/unittest/test_scalability.py 0.00% <0.00%> (-96.16%) ⬇️
python/graphscope/tests/unittest/test_app.py 0.00% <0.00%> (-95.66%) ⬇️
...hon/graphscope/tests/unittest/test_create_graph.py 0.00% <0.00%> (-92.00%) ⬇️
python/graphscope/tests/unittest/test_graph.py 0.00% <0.00%> (-84.93%) ⬇️
python/graphscope/tests/unittest/test_context.py 0.00% <0.00%> (-81.04%) ⬇️
... and 38 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 fa5718c...3dbef24. Read the comment docs.

@lidongze0629 lidongze0629 self-requested a review May 25, 2022 15:47
[Sprint 05.16-06.02] GraphScope New Interface Implementation & Benchmark Prepare automation moved this from Review in progress to Reviewer approved May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants