-
Notifications
You must be signed in to change notification settings - Fork 443
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
clear duplicated and unused code #777
Conversation
Co-authored-by: 海虾 <tianli.tl@alibaba-inc.com>
1. Update edge id from string to bytes in pb 2. Use edge inner id for v2 and vineyard as edge ID in gaia runtime 3. Convert edge id into i64 from bytes for v2 in compiler 4. Change permission of ${GITHUB_WORKSPACE} as the remnants of build-gss cannot be deleted Co-authored-by: BingqingLyu <lv_bingqing@163.com> Co-authored-by: siyuan0322 <siyuan0322@gmail.com>
* Integrate GraphManager into Coordinator * Do not create a pod during `sess.gremlin` * Avoid using root user in docker image * Unify GRAPHSCOPE_HOME prefix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please double check the tags for the basic images, make sure that these changes are essential.
I have no other comments! Thanks again for tidy up the codebase!
.github/workflows/ci.yml
Outdated
@@ -592,6 +593,7 @@ jobs: | |||
name: gss-${{ github.sha }} | |||
path: | | |||
maxgraph.tar.gz | |||
graphscope_store_data_load.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to add another upload-artifact
step to only load the graphscope_store_data_load.tar.gz
, or else it will be package together with maxgraph.tar.gz which has a much larger size, It will be convenient for us to be able to download this artifact only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
What do these changes do?
Related issue number
Fixes #536