Skip to content

[hackathon] chore: add probabilistic data structures and commands #5106

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

kostasrim
Copy link
Contributor

@kostasrim kostasrim commented May 12, 2025

Implements:

  • tdigest
  • topk

maybe cuckoo filter

* import t-digest-c lib

---------

Signed-off-by: kostas <kostas@dragonflydb.io>
@romange
Copy link
Collaborator

romange commented May 12, 2025

I prefer we keep this in a hackathon branch for now - so you will have freedom to experiment with half baked code.

@kostasrim
Copy link
Contributor Author

@romange This is the hackathon branch. It's the base of our project. We branch off this branch and we merge each of the PR's here. I was not planning to merge this branch in main (nothing that is developed here will without discussion moved to main branch).

kostasrim added 3 commits May 12, 2025 15:47
* chore: add tdigest to compact object

---------

Signed-off-by: kostas <kostas@dragonflydb.io>
Signed-off-by: kostas <kostas@dragonflydb.io>
* quantiles
* trimmed_mean
* merge

Signed-off-by: kostas <kostas@dragonflydb.io>
@kostasrim kostasrim changed the title [hackathon] chore: add probabilistic data structures [hackathon] chore: add probabilistic data structures and commands May 14, 2025
kostasrim added 2 commits May 15, 2025 10:32
* add support for topk ds in compact object
* add topk.reserve prototype
* add topk.add prototype
* add topk.list prototype

Signed-off-by: kostas <kostas@dragonflydb.io>
* add topk.query
* add topk.info
* add topk.incrby
* add tests
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

Successfully merging this pull request may close these issues.

3 participants