Skip to content

Conversation

facewise
Copy link

Hi.

I'd like to suggest support for Redis static Master/Replica auto-configuration.
This auto-configuration makes LettuceConnectionFactory use RedisStaticMasterReplicaConfiguration, e.g. AWS ElastiCache with Replicas.
I know it's cautious since Jedis doesn't support for static master-replica connection in Jedis, while Lettuce does.

Please feel free to review this suggestion and share your thoughts.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 26, 2025
@wilkinsona
Copy link
Member

Thanks for the PR.

Given that the static master-replica support is specific to Lettuce, I am not sure that the proposed additions should be as central to the auto-configuration as they currently are. For example, the properties could be nested beneath spring.data.redis.lettuce to indicate that they're only supported with Lettuce.

It also doesn't feel quite right for RedisConnectionDetails to know about a Lettuce-specific feature. I'm not sure how to address this as I don't think we've faced this problem and I don't think we have an established pattern for dealing with it.

@facewise
Copy link
Author

I agree. Would it still be consideration if the properties go beneath spring.data.redis.lettuce and make RedisConnectionDetails unaware of this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-triage An issue we've not yet triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants