Deprecate Remove ShardedJedisPipeline class #2500 (#2508)

* Deprecate Remove ShardedJedisPipeline class #2500 * putting back the tests
redis · Apr 14, 2021 · 4b1027b · 4b1027b
1 parent 4e6d56b
commit 4b1027b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/main/java/redis/clients/jedis/BinaryShardedJedis.java b/src/main/java/redis/clients/jedis/BinaryShardedJedis.java
@@ -875,6 +875,12 @@ public Long linsert(final byte[] key, final ListPosition where, final byte[] piv
     return j.linsert(key, where, pivot, value);
   }
 
+  /**
+   * @return
+   * @deprecated The support of {@link ShardedJedisPipeline} and this method will be removed in next
+   * major release.
+   */
+  @Deprecated
   public ShardedJedisPipeline pipelined() {
     ShardedJedisPipeline pipeline = new ShardedJedisPipeline();
     pipeline.setShardedJedis(this);

diff --git a/src/main/java/redis/clients/jedis/ShardedJedisPipeline.java b/src/main/java/redis/clients/jedis/ShardedJedisPipeline.java
@@ -5,6 +5,11 @@
 import java.util.List;
 import java.util.Queue;
 
+/**
+ * @deprecated The support of {@link ShardedJedisPipeline} will be removed in next major release,
+ * due to several bugs and implementation issue.
+ */
+@Deprecated
 public class ShardedJedisPipeline extends PipelineBase {
   private BinaryShardedJedis jedis;
   private List<FutureResult> results = new ArrayList<>();