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.

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

SCAN command for cluster mode - redis-rs impl' #986

Open
shachlanAmazon opened this issue Feb 18, 2024 · 0 comments
Open

SCAN command for cluster mode - redis-rs impl' #986

shachlanAmazon opened this issue Feb 18, 2024 · 0 comments
Assignees
Milestone

Comments

@shachlanAmazon
Copy link
Contributor

SCAN is complex to implement over cluster mode. We need some way to iterate over all the slots, without missing values due to topology changes / slot migrations.
We can take a hint from the ElastiCache serverless SCAN implementation.

Suggested high-level implementation -
since we're allowed to send the same value twice, we can keep a list of finished slots. We iterate over nodes serially. Once we scan over a node successfully, we'll mark all of its slots as finished. if a topology refresh happens while iterating over a node, we restart the process for all unfinished slots.

@shachlanAmazon shachlanAmazon added this to the GA milestone Feb 18, 2024
@avifenesh avifenesh changed the title SCAN command for cluster mode SCAN command for cluster mode - redis-rs impl' Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants