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

Retry once to many on `MOVED` error at cluster #882

Merged

Conversation

@haruwo
Copy link

haruwo commented Nov 21, 2019

Fix for the cluster sending recursive MOVED error in very short time.

In my load test (N million times) I had the following behavior.
When I send SET, REM, SADD and SREM to redis cluster on AWS Elastic Cache Redis cluster, sometime I detect a MOVED error on my code.
Maybe this is because redis slots/node mapping was updated in a very short time. Anyway, I changed retry once to many time on MOVED error and it solved the problem.

@byroot byroot merged commit bd6ee65 into redis:master Nov 24, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.