Skip to content

Conversation

@joshrotenberg
Copy link
Contributor

Documentation additions for UnifiedJedis.

Comment on lines 89 to 92
* @see JedisPool For connection pooling with standalone Redis servers
* @see JedisCluster For dedicated Redis Cluster support
* @see JedisSentineled For dedicated Redis Sentinel support
* @see JedisSentinelPool For Redis Sentinel connection pooling
Copy link
Collaborator

Choose a reason for hiding this comment

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

JedisPool & JedisSentinelPool are managing Jedis client ( I mean not UnifiedJedis), I think we should remove them from here and refer them in Jedis itself

@ggivo
Copy link
Collaborator

ggivo commented Jun 16, 2025

Hi @joshrotenberg
I see that the review is marked as Draft. Do you plan further changes?
Seems there are also some conflicts to be resolved

@joshrotenberg joshrotenberg deleted the docs/unified-jedis-constructors branch July 5, 2025 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants