-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
subscribe never failed when sentinel master switched #583
Comments
What currently happens is that go-redis tries to close free connections to the old master. But PubSub connections are never free and therefore that connection is never closed. I think the proper fix would be to close all connections after master address is changed. |
Sentinel should be fixed in v6.5.1. Keeping this open as reminder to do something similar for Redis Cluster |
Somehow I missed this when it occurred -- thank you, 6 months late. |
I've similar issue (should I open another?), when Redis Cluster failed beside blocking and non-blocking operations, and some master changes its state:
The cluster resurrects, but several connections to the old masters stays established. |
It makes sense to open new issue if you have a small program that reproduces the problem. This issue is about sentinel which is completely different software. |
This issue is marked stale. It will be closed in 30 days if it is not updated. |
Looking for thoughts here -- it's possible I'm mis-using a portion of the library. I've got a go program that ties two different redis instances together via pub/sub:
.. the other day we had a sentinel failover in Redis A; the event count dropped to 0, no events were being published to B, cue disaster music. The particular redis instance we were connected to never died, but no events were being published to it after sentinel failover.
The library may well -- probably did -- receive the +switch-master notification, but that never bubbled up to me, and the existing connection was happy.
How would you handle this case? Is there an intention to bubble sentinel events up to clients in some way?
The text was updated successfully, but these errors were encountered: