-
Notifications
You must be signed in to change notification settings - Fork 54
Redis Stack use cases #65
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
Conversation
✅ Deploy Preview for redis-stack-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, didn't review the content :) Perhaps @K-Jo should be the final reviewer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initial review
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
…s-stack-docs into use-cases-topic
…s-stack-docs into use-cases-topic
@LiorKogan, please review one more time. I've added t-digest and responded to your comments. cc @banker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed that some probabilistic data structures are mentioned by name (HyperLogLog, TopK) but others do not. I think we should name them all.
I think it would be better to list the different probabilistic data structures and the questions each can answer, and then give a separate set of use cases for each. Otherwise, it is a little bit confusing.
I listed those questions for all data structures in https://redislabs.atlassian.net/wiki/spaces/DX/pages/3620077569/t-digest+A+New+Probabilistic+Data+Structure+in+RedisBloom.
You can copy from there if you wish.
I also think we should include HyperLogLog because it is also a probabilistic data structure, but mention that it is included in core Redis.
@LiorKogan I've added back the headings for probabilistic data structures. Mind checking it again? Thank you so much! |
Looks good! |
@banker, please review so I can merge. Thank you! Many thanks, @LiorKogan! |
Adds new section in the Redis Stack category