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

Support TDIGEST.MERGESTORE and make compression optional on TDIGEST.CREATE #2216

Closed
chayim opened this issue Jul 27, 2022 · 1 comment · Fixed by #2214
Closed

Support TDIGEST.MERGESTORE and make compression optional on TDIGEST.CREATE #2216

chayim opened this issue Jul 27, 2022 · 1 comment · Fixed by #2214
Labels
feature

Comments

@chayim
Copy link
Contributor

chayim commented Jul 27, 2022

New changes for the RedisBloom 2.4 milestone mean we have API changes to make prior to the next milestone.
@filipecosta90
Copy link

The new command has been merged and documentation updated to include it.
More info in:
https://redis.io/commands/tdigest.mergestore/

@chayim chayim added feature and removed feature labels Jul 28, 2022
leibale added a commit to leibale/node-redis that referenced this issue Jul 28, 2022
@leibale
close redis#2216 - add support for TDIGEST.MERGESTORE and make compre…
8c1c28d 
…ssion optional on TDIGEST.CREATE
@leibale leibale mentioned this issue Oct 27, 2022
Merged
leibale added a commit that referenced this issue Nov 1, 2022
@leibale
Add support for T-Digest (#2214)
be90e62 
* wip

* close #2216 - add support for TDIGEST.MERGESTORE and make compression optional on TDIGEST.CREATE

* fix some tdigest commands, use bloom edge docker

* fix index.ts

* 2.4-RC2 (v2.4.1)

* fix some commands and tests

* clean code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for T-Digest leibale/node-redis
2 participants
@chayim @filipecosta90