Skip to content
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

PlainTable hard codes num_probes=6 for Bloom filter, regardless of bloom_bits_per_key #5743

Open
pdillinger opened this issue Aug 26, 2019 · 2 comments

Comments

@pdillinger
Copy link
Contributor

Expected behavior

When using PlainTable with Bloom filter, expect it to choose reasonable num_probes based on bloom_bits_per_key setting.

Actual behavior

Always uses num_probes = 6 even when that's really bad for speed and false positive rate.

Steps to reproduce the behavior

Just based on code inspection:

table_builder_options.column_family_name, 6,

@pdillinger
Copy link
Contributor Author

And here:

@liam-thunder
Copy link
Contributor

I prepared a PR for this issue
#5833

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants