Deprecate JedisPool and JedisSentinelPool

Provide clear guidance which client class should be used instead.

Author: uglide

src/main/java/redis/clients/jedis/Jedis.java

@@ -32,6 +32,62 @@
 import redis.clients.jedis.util.KeyValue;
 import redis.clients.jedis.util.Pool;
 
+/**
+ * Jedis is a lightweight Redis client that uses a single, non-pooled connection to Redis.
+ * <p>
+ * <b>Important:</b> For most production use cases, {@link RedisClient} is the recommended and
+ * preferred option. {@code RedisClient} provides connection pooling, better resource management,
+ * and improved performance for typical applications.
+ * </p>
+ * <p>
+ * <b>When to use Jedis:</b>
+ * </p>
+ * <ul>
+ * <li><b>Short-lived scripts or utilities:</b> When you need a simple, lightweight client for
+ * one-off operations or command-line tools.</li>
+ * <li><b>Testing and development:</b> For unit tests or local development where connection pooling
+ * overhead is unnecessary.</li>
+ * <li><b>Fine-grained connection control:</b> Advanced scenarios requiring explicit control over
+ * individual connections, such as managing connection lifecycle manually or implementing custom
+ * connection strategies.</li>
+ * <li><b>Single-threaded applications:</b> Applications that execute Redis commands sequentially
+ * from a single thread and don't benefit from connection pooling.</li>
+ * </ul>
+ * <p>
+ * <b>When to use RedisClient instead:</b>
+ * </p>
+ * <ul>
+ * <li><b>Production applications:</b> Any multi-threaded or high-throughput application should use
+ * {@link RedisClient} for its connection pooling capabilities.</li>
+ * <li><b>Web applications:</b> Server applications handling concurrent requests benefit from
+ * connection pooling to avoid connection overhead.</li>
+ * <li><b>Long-running services:</b> Applications that maintain persistent connections to Redis
+ * should use {@link RedisClient} for better resource management.</li>
+ * <li><b>Default choice:</b> If you're unsure which to use, choose {@link RedisClient}.</li>
+ * </ul>
+ * <p>
+ * <b>Usage example:</b>
+ * </p>
+ *
+ * <pre>
+ * {
+ *   &#64;code
+ *   // Simple usage for a short-lived operation
+ *   try (Jedis jedis = new Jedis("localhost", 6379)) {
+ *     jedis.set("key", "value");
+ *     String value = jedis.get("key");
+ *   }
+ * }
+ * </pre>
+ * <p>
+ * <b>Note:</b> Each {@code Jedis} instance maintains a single connection. For concurrent access
+ * from multiple threads, either use {@link RedisClient} with connection pooling, or create
+ * separate {@code Jedis} instances per thread (not recommended for production).
+ * </p>
+ *
+ * @see RedisClient for the recommended pooled client for production use
+ * @see JedisPool for legacy pooled connections (deprecated, use RedisClient instead)
+ */
 public class Jedis implements ServerCommands, DatabaseCommands, JedisCommands, JedisBinaryCommands,
     ControlCommands, ControlBinaryCommands, ClusterCommands, ModuleCommands, GenericControlCommands,
     SentinelCommands, CommandCommands,  Closeable {

src/main/java/redis/clients/jedis/JedisFactory.java

@@ -17,9 +17,19 @@
 import redis.clients.jedis.util.JedisURIHelper;
 
 /**
- * PoolableObjectFactory custom impl.
+ * PooledObjectFactory implementation for creating and managing {@link Jedis} instances in connection pools.
+ * <p>
+ * This factory is used internally by {@link JedisPool} and {@link JedisSentinelPool} to create, validate,
+ * and destroy pooled Jedis connections.
+ * </p>
+ *
+ * @deprecated JedisFactory is used exclusively with the deprecated {@link JedisPool} and {@link JedisSentinelPool}
+ *             classes. For modern Redis clients ({@link RedisClient}, {@link RedisSentinelClient}), the framework
+ *             uses {@link ConnectionFactory} internally, which manages {@link Connection} objects instead of
+ *             {@link Jedis} instances. There is no direct replacement for JedisFactory as connection management
+ *             is handled automatically by the new client architecture.
  */
-// Legacy
+@Deprecated
 public class JedisFactory implements PooledObjectFactory<Jedis> {
 
   private static final Logger logger = LoggerFactory.getLogger(JedisFactory.class);

src/main/java/redis/clients/jedis/JedisPool.java

@@ -13,7 +13,14 @@
 import redis.clients.jedis.util.JedisURIHelper;
 import redis.clients.jedis.util.Pool;
 
-// Legacy
+/**
+ * JedisPool is a pooled connection client for standalone Redis servers.
+ *
+ * @deprecated Use {@link RedisClient} instead. RedisClient provides the same functionality
+ *             with a cleaner API and simplified constructor options. For basic usage, simple
+ *             constructors are available. For advanced configuration, use {@link RedisClient#builder()}.
+ */
+@Deprecated
 public class JedisPool extends Pool<Jedis> {
 
   private static final Logger log = LoggerFactory.getLogger(JedisPool.class);

src/main/java/redis/clients/jedis/JedisPoolConfig.java

@@ -3,6 +3,15 @@
 import java.time.Duration;
 import org.apache.commons.pool2.impl.GenericObjectPoolConfig;
 
+/**
+ * Configuration class for {@link JedisPool} connection pooling.
+ *
+ * @deprecated JedisPoolConfig is used with the deprecated {@link JedisPool} and {@link JedisSentinelPool} classes.
+ *             Use {@link ConnectionPoolConfig} instead, which is designed for the modern {@link RedisClient}
+ *             and {@link RedisSentinelClient} classes. ConnectionPoolConfig provides the same pooling configuration
+ *             options with better integration into the new client architecture.
+ */
+@Deprecated
 public class JedisPoolConfig extends GenericObjectPoolConfig<Jedis> {
 
   public JedisPoolConfig() {

src/main/java/redis/clients/jedis/JedisSentinelPool.java

@@ -18,6 +18,14 @@
 import redis.clients.jedis.exceptions.JedisException;
 import redis.clients.jedis.util.Pool;
 
+/**
+ * JedisSentinelPool is a pooled connection client for Redis Sentinel deployments.
+ *
+ * @deprecated Use {@link RedisSentinelClient} instead. RedisSentinelClient provides the same functionality
+ *             with a cleaner API and simplified constructor options. For basic usage, simple
+ *             constructors are available. For advanced configuration, use {@link RedisSentinelClient#builder()}.
+ */
+@Deprecated
 public class JedisSentinelPool extends Pool<Jedis> {
 
   private static final Logger LOG = LoggerFactory.getLogger(JedisSentinelPool.class);