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

Implemented read-write separation based Sentinel #3797

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Vampirea
Copy link

@Vampirea Vampirea commented Apr 3, 2024

Implemented read-write separation function based on Redis Sentinel, balancing read request load to all slave, Support Redis high availability and failover.

Copy link
Collaborator

@sazzad16 sazzad16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all thank you for using Jedis and for trying to improve it.

  1. The term slave is either removed or deprecated.
  2. Currently our focus is more towards UnifiedJedis (with the help of CommandExecutor, ConnectionProvider, etc.) compared to Jedis and Pool. So, at this moment, whether you implement this with Jedis and Pool or not, we prefer an implementation with UnifiedJedis.
  3. It would be great if this can be done in JedisSentineled with some configs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants