-
Notifications
You must be signed in to change notification settings - Fork 455
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
feat: support cuckoo filter and most commands #2545
base: unstable
Are you sure you want to change the base?
Conversation
…into cuckoo-filter
Running |
@jonathanc-n Thank you for this contribution. Great job! |
Thanks for your contribution! However, I cannot find a proposal to state the design details and storage format of the new data structure. May I ask you provide such a proposal before we continue to review this patch? |
@PragmaTwice it is mentioned in #2539, I added some extra information to clarify some things. |
references issue: #2534
Added support for all commands except for SCANDUMP and LOADCHUNK.
The go cases are currently being worked on.