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.

Sign up for GitHub

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

Jump to bottom

TDigest: Implement MIN MAX command for TDigest Algorithm #2793

Open
2 tasks done
Tracked by #2558
LindaSummer opened this issue Feb 15, 2025 · 2 comments · May be fixed by #2811
Open
2 tasks done
Tracked by #2558

TDigest: Implement MIN MAX command for TDigest Algorithm #2793

LindaSummer opened this issue Feb 15, 2025 · 2 comments · May be fixed by #2811
Assignees
@wyxxxcat
Labels
enhancement type enhancement

Comments

@LindaSummer
Copy link
Contributor

LindaSummer commented Feb 15, 2025 •
edited
Loading

Search before asking

  • I had searched in the issues and found no similar issues.

Motivation

Refer: #2558
We should implement TDIGEST.MIN TDIGEST.MAX command.

Redis command reference:

Solution

Depends on #2791

We can follow the implementation of #2791 for this command.

To write more comprehensive integration tests, it will be better to be implemented after #2792 .

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@LindaSummer LindaSummer added the enhancement type enhancement label Feb 15, 2025
@LindaSummer LindaSummer mentioned this issue Feb 14, 2025
Open
17 tasks
@wyxxxcat
Copy link
Contributor

I want to try, please assign it to me, thx

@PragmaTwice
Copy link
Member

Assigned. Feel free to open PRs!

@git-hulk git-hulk linked a pull request Feb 28, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wyxxxcat wyxxxcat

Labels
enhancement type enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(tdigest): add TDIGEST.MAX/MIN command wyxxxcat/kvrocks
3 participants
@LindaSummer @PragmaTwice @wyxxxcat