Merge pull request #2995 from dengliming/f_calcslot_bug

Fix ClusterConnectionManager#calcSlot bug.

redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java

@@ -718,8 +718,10 @@ public class ClusterConnectionManager extends MasterSlaveConnectionManager {
         int start = indexOf(key, (byte) '{');
         if (start != -1) {
             int end = indexOf(key, (byte) '}');
+            if (end != -1 && start + 1 < end) {
                 key = Arrays.copyOfRange(key, start + 1, end);
             }
+        }
         
         int result = CRC16.crc16(key) % MAX_SLOT;
         return result;
@@ -734,8 +736,10 @@ public class ClusterConnectionManager extends MasterSlaveConnectionManager {
         int start = key.indexOf('{');
         if (start != -1) {
             int end = key.indexOf('}');
+            if (end != -1 && start + 1 < end) {
                 key = key.substring(start + 1, end);
             }
+        }
 
         int result = CRC16.crc16(key.getBytes()) % MAX_SLOT;
         log.debug("slot {} for {}", result, key);

redisson/src/test/java/org/redisson/RedissonSetTest.java

@@ -530,7 +530,7 @@ public class RedissonSetTest extends BaseTest {
         RedissonClient redisson = Redisson.create(config);
 
         int size = 10000;
-        RSet<String> set = redisson.getSet("test");
+        RSet<String> set = redisson.getSet("{test");
         for (int i = 0; i < size; i++) {
             set.add("" + i);
         }