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
Implement Entry API for storage2::LazyHashMap #480
Conversation
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.
Just looked at the big picture so far not in detail.
Codecov Report
@@ Coverage Diff @@
## master #480 +/- ##
==========================================
+ Coverage 85.08% 85.89% +0.81%
==========================================
Files 127 128 +1
Lines 5497 5800 +303
==========================================
+ Hits 4677 4982 +305
+ Misses 820 818 -2
Continue to review full report at Codecov.
|
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.
looks good overall, some nits
383b791
to
3869bc6
Compare
3869bc6
to
d64972c
Compare
6d3fdcf
to
5e4bdd5
Compare
5e4bdd5
to
758643c
Compare
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.
LGTM
Closes #479.
Current benchmarks
Before
HashMap
was using the newLazyHashMap
Entry APINow that
HashMap
is using the newLazyHashMap
Entry API