RedisNodes ping results check optimization

Signed-off-by: blackstorm <inf2inf2@outlook.com>
4 years ago · 5b938edd2f
parent 78c60bf65b
commit 5b938edd2f
1 changed files with 4 additions and 2 deletions
--- a/redisson/src/main/java/org/redisson/RedisNodes.java
+++ b/redisson/src/main/java/org/redisson/RedisNodes.java
@ -149,10 +149,12 @@ public class RedisNodes<N extends Node> implements NodesGroup<N> {
        for (Entry<RedisConnection, RFuture<String>> entry : result.entrySet()) {
            RFuture<String> f = entry.getValue();
            f.awaitUninterruptibly();
-            if (!"PONG".equals(f.getNow())) {
+            String pong = f.getNow();
+            entry.getKey().closeAsync();
+            if (!"PONG".equals(pong)) {
                res = false;
+                break;
            }
-            entry.getKey().closeAsync();
        }

        // true and no futures were missed during client connection