Fix redis#550 : Removed redundant call to connectionPool.getResource() in method getConnectionFromSlot(int slot) in class JedisSlotBasedConnectionHandler

Mayank Dang · Mayank Dang · commit c4fb2b4adca7 · 2014-02-17T16:52:19.000+05:30
diff --git a/.gitignore b/.gitignore
@@ -9,3 +9,6 @@ target/
 build/
 bin/
 tags
+.idea
+*.aof
+*.rdb
diff --git a/src/main/java/redis/clients/jedis/JedisSlotBasedConnectionHandler.java b/src/main/java/redis/clients/jedis/JedisSlotBasedConnectionHandler.java
@@ -40,7 +40,7 @@ public Jedis getConnectionFromSlot(int slot) {
 	    connectionPool = getRandomConnection();
 	}
 	currentConnection = connectionPool.getResource();
-	return connectionPool.getResource();
+	return currentConnection;
     }
 
 }

-Original file line number
+Diff line change
 build/
 bin/
 tags
 +.idea
 +*.aof
 +*.rdb
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ public Jedis getConnectionFromSlot(int slot) {`
`40`	`40`	`connectionPool = getRandomConnection();`
`41`	`41`	`}`
`42`	`42`	`currentConnection = connectionPool.getResource();`
`43`		`- return connectionPool.getResource();`
	`43`	`+ return currentConnection;`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`}`