From 2b1cb8fe318301b4726b495f3808b249a1fece34 Mon Sep 17 00:00:00 2001 From: Jungtaek Lim Date: Sat, 27 Sep 2014 15:41:13 +0900 Subject: [PATCH] Fix zrangebylex with pipelined (parameters reversed) --- .../redis/clients/jedis/BinaryRedisPipeline.java | 4 ++-- src/main/java/redis/clients/jedis/PipelineBase.java | 12 ++++++------ src/main/java/redis/clients/jedis/RedisPipeline.java | 4 ++-- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/main/java/redis/clients/jedis/BinaryRedisPipeline.java b/src/main/java/redis/clients/jedis/BinaryRedisPipeline.java index 98ebc73..f8a60c0 100644 --- a/src/main/java/redis/clients/jedis/BinaryRedisPipeline.java +++ b/src/main/java/redis/clients/jedis/BinaryRedisPipeline.java @@ -209,9 +209,9 @@ public interface BinaryRedisPipeline { Response zlexcount(final byte[] key, final byte[] min, final byte[] max); - Response> zrangeByLex(final byte[] key, final byte[] max, final byte[] min); + Response> zrangeByLex(final byte[] key, final byte[] min, final byte[] max); - Response> zrangeByLex(final byte[] key, final byte[] max, final byte[] min, + Response> zrangeByLex(final byte[] key, final byte[] min, final byte[] max, int offset, int count); Response zremrangeByLex(final byte[] key, final byte[] min, final byte[] max); diff --git a/src/main/java/redis/clients/jedis/PipelineBase.java b/src/main/java/redis/clients/jedis/PipelineBase.java index 58d06e1..3439323 100644 --- a/src/main/java/redis/clients/jedis/PipelineBase.java +++ b/src/main/java/redis/clients/jedis/PipelineBase.java @@ -1039,27 +1039,27 @@ abstract class PipelineBase extends Queable implements BinaryRedisPipeline, } @Override - public Response> zrangeByLex(final byte[] key, final byte[] max, final byte[] min) { + public Response> zrangeByLex(final byte[] key, final byte[] min, final byte[] max) { getClient(key).zrangeByLex(key, min, max); return getResponse(BuilderFactory.BYTE_ARRAY_ZSET); } @Override - public Response> zrangeByLex(final String key, final String max, final String min) { + public Response> zrangeByLex(final String key, final String min, final String max) { getClient(key).zrangeByLex(key, min, max); return getResponse(BuilderFactory.STRING_ZSET); } @Override - public Response> zrangeByLex(final byte[] key, final byte[] max, - final byte[] min, final int offset, final int count) { + public Response> zrangeByLex(final byte[] key, final byte[] min, + final byte[] max, final int offset, final int count) { getClient(key).zrangeByLex(key, min, max, offset, count); return getResponse(BuilderFactory.BYTE_ARRAY_ZSET); } @Override - public Response> zrangeByLex(final String key, final String max, - final String min, final int offset, final int count) { + public Response> zrangeByLex(final String key, final String min, + final String max, final int offset, final int count) { getClient(key).zrangeByLex(key, min, max, offset, count); return getResponse(BuilderFactory.STRING_ZSET); } diff --git a/src/main/java/redis/clients/jedis/RedisPipeline.java b/src/main/java/redis/clients/jedis/RedisPipeline.java index 63e3257..d518195 100644 --- a/src/main/java/redis/clients/jedis/RedisPipeline.java +++ b/src/main/java/redis/clients/jedis/RedisPipeline.java @@ -187,9 +187,9 @@ public interface RedisPipeline { Response zlexcount(final String key, final String min, final String max); - Response> zrangeByLex(final String key, final String max, final String min); + Response> zrangeByLex(final String key, final String min, final String max); - Response> zrangeByLex(final String key, final String max, final String min, + Response> zrangeByLex(final String key, final String min, final String max, final int offset, final int count); Response zremrangeByLex(final String key, final String start, final String end);