-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
Improve FullFilterBitsReader::HashMayMatch's doc #4202
Conversation
It should be change to this:
Do you have time to apply this change? And also piggyback the minor fixes mentioned in these PRs? |
6fb5ead
to
8bba425
Compare
@maysamyabandeh A new commit which includes the suggested PRs is submitted. Please take a look. |
util/bloom.cc
Outdated
// FilterBitsBuilder::Finish. This method must return true if the key was | ||
// passed to FilterBitsBuilder::AddKey. This method may return true or false | ||
// if the key was not on the list, but it should aim to return false with a | ||
// high probability. |
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.
Can you remove the trailing spaces?
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.
- Remove the unused variable write_context. - Fix two typos. - Improve HashMayMatch function's comment.
8bba425
to
8d6bbf2
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.
maysamyabandeh is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: HashMayMatch is related to AddKey() instead of CreateFilter(). Also applies some minor Fixes facebook#4191 facebook#4200 facebook#3910 Pull Request resolved: facebook#4202 Differential Revision: D9180945 Pulled By: maysamyabandeh fbshipit-source-id: 6f07b81c5bb9bda5c0273475b486ba8a030471e6
HashMayMatch is related to AddKey() instead of CreateFilter().
Also applies some minor Fixes #4191 #4200 #3910