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
Enable perfect hash support in GAE #2959
Conversation
@@ -265,7 +268,8 @@ def __init__( | |||
Defaults to global. | |||
compact_edges (bool, optional): Compact edges (CSR) using varint and delta encoding. Defaults to False. | |||
Note that compact edges helps to half the memory usage of edges in graph data structure, but may cause | |||
at most 10%~20% performance degeneration in some algorithms. | |||
at most 10%~20% performance degeneration in some algorithms. Defaults to False. | |||
use_perfect_hash (bool, optional): Use perfect hash in vertex map to optimize the memory usage. Defaults to False. |
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.
Add a comment saying it only works for oid type int64_t? It I understand correctly.
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.
Or just check the oid and log error.
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.
Works for string oid as well.
4da94ee
to
eccc5db
Compare
Signed-off-by: Tao He <linzhu.ht@alibaba-inc.com>
eccc5db
to
ba1b528
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2959 +/- ##
===========================================
+ Coverage 42.40% 73.31% +30.91%
===========================================
Files 99 99
Lines 10699 10733 +34
===========================================
+ Hits 4537 7869 +3332
+ Misses 6162 2864 -3298
... and 50 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
No description provided.