socketry / async-redis

MIT License
83 stars 18 forks source link

Support for Redis Sentinels #28

Closed davidor closed 4 years ago

davidor commented 4 years ago

Are there any plans to add support for Redis sentinels? If there aren't, @ioquatix would you be open to accept contributions for this?

Refs: https://redis.io/topics/sentinel https://github.com/redis/redis-rb#sentinel-support

ioquatix commented 4 years ago

What kind of design makes the most sense to you, considering the current implementation?

davidor commented 4 years ago

Not sure yet. I'll try to find some time in the next days to experiment a bit. For now, I've just adapted some code from redis-rb to resolve masters/slaves by asking the sentinels: https://github.com/davidor/async-redis/tree/sentinels