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

Add support for Redis Sentinel #17

Open
mikn opened this issue Apr 22, 2021 · 0 comments
Open

Add support for Redis Sentinel #17

mikn opened this issue Apr 22, 2021 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@mikn
Copy link
Owner

mikn commented Apr 22, 2021

With PubSub support this should be relatively easy to add as Redis Sentinel is mostly a group of servers that publishes the current quorum master over a topic.

Documentation: https://redis.io/topics/sentinel

In a nutshell, the Sentinel implementation should subscribe to all sentinels provided in the connection string, subscribe to the relevant topics, listen for updates and update the "real master" in the connection. The easiest implementation is probably to provide some kind of wrapper for the Pool object that returns connections that are according to the sentinels correct at the time. We may also need to wrap the connection object itself to provide for checks and proper failover behaviour.

@mikn mikn added the enhancement New feature or request label Apr 22, 2021
@mikn mikn added this to the 0.2.0 milestone Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant