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

Control Reliability not using autoreconnect. #1309

Closed
doo871128 opened this issue Jun 14, 2020 · 1 comment
Closed

Control Reliability not using autoreconnect. #1309

doo871128 opened this issue Jun 14, 2020 · 1 comment
Labels
status: duplicate A duplicate of another issue

Comments

@doo871128
Copy link

Feature Request

Is your feature request related to a problem? Please describe

For now, we are using lettuce to handle large scales of traffic. The traffic that comes to REDIS cluster per minute is more than 10 million per minute. In this situation, there are some cases where REDIS went down abruptly with various reasons(hardware failure, instance restart etc). The problem that we are facing is when the restarted REDIS instance is attached to REDIS cluster, there are a big CPU spikes causing latency spikes. It turns out that when REDIS instance goes back to REDIS cluster, lettuce client try to send the failed command to the recovered REDIS INSTANCE using AT LEAST ONCE mode of reliability. After changing auto reconnect mode to false,
the latency spike problem is gone. But, another problem that we have is we cannot reconnect that recovered REDIS instance. This is because only way to change reliability is to change auto connect mode. So, my suggestion/question is
why can't we set the only reliability level regardless of auto connect mode?
Is there any reason why reliability level should follow the auto reconnect mode?

A clear and concise description of what the problem is. Ex. I have an issue when [...]

Describe the solution you'd like

I want to suggest that if we can adjust the reliability level creating another option like RELIABILITY.
That can solve my problem.

A clear and concise description of what you want to happen. Add any considered drawbacks.
For now, I didn't understand why this RELIABILITY has to follow the auto reconnect mode.

@doo871128 doo871128 added the type: feature A new feature label Jun 14, 2020
@mp911de
Copy link
Collaborator

mp911de commented Jun 24, 2020

Closing this one as duplicate of #1310.

@mp911de mp911de closed this as completed Jun 24, 2020
@mp911de mp911de added status: duplicate A duplicate of another issue and removed type: feature A new feature labels Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: duplicate A duplicate of another issue
Projects
None yet
Development

No branches or pull requests

2 participants