diff --git a/redisson/src/main/java/org/redisson/connection/MasterSlaveEntry.java b/redisson/src/main/java/org/redisson/connection/MasterSlaveEntry.java index 3dba2f196..16904068d 100644 --- a/redisson/src/main/java/org/redisson/connection/MasterSlaveEntry.java +++ b/redisson/src/main/java/org/redisson/connection/MasterSlaveEntry.java @@ -180,9 +180,6 @@ public class MasterSlaveEntry { if (!config.isSlaveNotUsed()) { addSlaveEntry(masterEntry); } - - masterConnectionPool.addEntry(masterEntry); - masterPubSubConnectionPool.addEntry(masterEntry); return client; }); }).whenComplete((r, e) -> { @@ -537,8 +534,6 @@ public class MasterSlaveEntry { } private void removeMaster(ClientConnectionsEntry masterEntry) { - masterConnectionPool.removeEntry(masterEntry); - masterPubSubConnectionPool.removeEntry(masterEntry); removeSlaveEntry(masterEntry); masterEntry.nodeDown(); masterEntry.shutdownAsync(); diff --git a/redisson/src/main/java/org/redisson/connection/pool/ConnectionPool.java b/redisson/src/main/java/org/redisson/connection/pool/ConnectionPool.java index 4907fd41d..75127c8e2 100644 --- a/redisson/src/main/java/org/redisson/connection/pool/ConnectionPool.java +++ b/redisson/src/main/java/org/redisson/connection/pool/ConnectionPool.java @@ -46,7 +46,7 @@ abstract class ConnectionPool { private final Logger log = LoggerFactory.getLogger(getClass()); - protected final Queue entries = new ConcurrentLinkedQueue<>(); + final Queue entries = new ConcurrentLinkedQueue<>(); final ConnectionManager connectionManager; diff --git a/redisson/src/main/java/org/redisson/connection/pool/MasterConnectionPool.java b/redisson/src/main/java/org/redisson/connection/pool/MasterConnectionPool.java index e0c954583..448c9fc25 100644 --- a/redisson/src/main/java/org/redisson/connection/pool/MasterConnectionPool.java +++ b/redisson/src/main/java/org/redisson/connection/pool/MasterConnectionPool.java @@ -33,6 +33,7 @@ import java.util.concurrent.CompletableFuture; * */ public class MasterConnectionPool extends ConnectionPool { + public MasterConnectionPool(MasterSlaveServersConfig config, ConnectionManager connectionManager, MasterSlaveEntry masterSlaveEntry) { super(config, connectionManager, masterSlaveEntry); @@ -48,7 +49,7 @@ public class MasterConnectionPool extends ConnectionPool { @Override public CompletableFuture get(RedisCommand command, boolean trackChanges) { - return acquireConnection(command, entries.peek(), trackChanges); + return acquireConnection(command, masterSlaveEntry.getEntry(), trackChanges); } } diff --git a/redisson/src/main/java/org/redisson/connection/pool/MasterPubSubConnectionPool.java b/redisson/src/main/java/org/redisson/connection/pool/MasterPubSubConnectionPool.java index c0651f4c8..e39fa2b1b 100644 --- a/redisson/src/main/java/org/redisson/connection/pool/MasterPubSubConnectionPool.java +++ b/redisson/src/main/java/org/redisson/connection/pool/MasterPubSubConnectionPool.java @@ -38,7 +38,7 @@ public class MasterPubSubConnectionPool extends PubSubConnectionPool { @Override public CompletableFuture get(RedisCommand command, boolean trackChanges) { - return acquireConnection(command, entries.peek(), trackChanges); + return acquireConnection(command, masterSlaveEntry.getEntry(), trackChanges); } }