Skip to content

fix(bloom): optimize the calculation of k #1763

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

Closed
wants to merge 1 commit into from

Conversation

chenat9
Copy link

@chenat9 chenat9 commented Dec 10, 2021

Accroding to theWIKIPEDIA, optimized the calculation of k (the number of hash functions) in bloomFilter


This change is Reviewable

@chenat9 chenat9 requested a review from manishrjain as a code owner December 10, 2021 09:18
@CLAassistant
Copy link

CLAassistant commented Dec 10, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


chencunge seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/stale The issue hasn't had activity for a while and it's marked for closing. label Apr 16, 2022
@stale
Copy link

stale bot commented Apr 27, 2022

This issue was marked as stale and no activity has occurred since then, therefore it will now be closed. Please, reopen if the issue is still relevant.

@stale stale bot closed this Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/stale The issue hasn't had activity for a while and it's marked for closing.
Development

Successfully merging this pull request may close these issues.

2 participants