GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

[OSPP] Spark RDD Reader for GraphScope

Open Issac-Newton opened this issue 3 years ago • 2 comments

What do these changes do?

Using gRPC to transfer data in GraphX(RDD) to GraphScope

Related issue number

#1529

Fixes

Issac-Newton avatar Sep 29 '22 13:09 Issac-Newton

Thanks for submitting your first pull request! You are awesome! 🤗 Please make sure you have signed the CLA, as this is a mandatory check before a PR being merged. Welcome to the GraphScope community! 🎉 You can meet the community on DingTalk or Slack.

welcome[bot] avatar Sep 29 '22 13:09 welcome[bot]

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 29 '22 14:09 CLAassistant

Hi, thank you for opening pull request for GraphScope! Could you please rebase your branch to main?

zhanglei1949 avatar Oct 28 '22 03:10 zhanglei1949

hi, @Issac-Newton, Please check your coding style(c++, java) to pass CI test. https://github.com/alibaba/GraphScope/blob/main/.github/workflows/docs.yml#L21 https://github.com/alibaba/GraphScope/blob/main/.github/workflows/docs.yml#L58

zhanglei1949 avatar Oct 28 '22 06:10 zhanglei1949

Codecov Report

Merging #2103 (bd8ff48) into main (74af74f) will decrease coverage by 5.86%. The diff coverage is n/a.

:exclamation: Current head bd8ff48 differs from pull request most recent head 6d8e869. Consider uploading reports for the commit 6d8e869 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2103      +/-   ##
==========================================
- Coverage   72.93%   67.07%   -5.87%     
==========================================
  Files          89       89              
  Lines        9789     9789              
==========================================
- Hits         7140     6566     -574     
- Misses       2649     3223     +574     
Impacted Files Coverage Δ
...aphscope/tests/kubernetes/test_resource_builder.py 0.00% <0.00%> (-100.00%) :arrow_down:
...on/graphscope/tests/kubernetes/test_demo_script.py 0.00% <0.00%> (-91.75%) :arrow_down:
python/graphscope/deploy/kubernetes/utils.py 11.90% <0.00%> (-58.58%) :arrow_down:
python/graphscope/deploy/kubernetes/cluster.py 23.01% <0.00%> (-51.70%) :arrow_down:
...thon/graphscope/tests/kubernetes/test_with_mars.py 0.00% <0.00%> (-44.83%) :arrow_down:
...n/graphscope/deploy/kubernetes/resource_builder.py 30.52% <0.00%> (-26.32%) :arrow_down:
python/graphscope/interactive/query.py 71.95% <0.00%> (-17.08%) :arrow_down:
python/graphscope/client/rpc.py 77.09% <0.00%> (-9.17%) :arrow_down:
python/graphscope/tests/conftest.py 76.53% <0.00%> (-4.43%) :arrow_down:
python/graphscope/client/session.py 73.62% <0.00%> (-2.85%) :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 25ab466...6d8e869. Read the comment docs.

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