fireChannelActive should be invoked on successful connection establishment

8 years ago · 75f8dea252
parent e354c4a135
commit 75f8dea252
1 changed files with 2 additions and 1 deletions
--- a/redisson/src/main/java/org/redisson/client/handler/BaseConnectionHandler.java
+++ b/redisson/src/main/java/org/redisson/client/handler/BaseConnectionHandler.java
@ -82,6 +82,7 @@ public abstract class BaseConnectionHandler<C extends RedisConnection> extends C
        }
        
        if (futures.isEmpty()) {
+            ctx.fireChannelActive();
            connectionPromise.trySuccess(connection);
            return;
        }
@ -97,7 +98,7 @@ public abstract class BaseConnectionHandler<C extends RedisConnection> extends C
                        return;
                    }
                    if (commandsCounter.decrementAndGet() == 0) {
-                        BaseConnectionHandler.super.channelActive(ctx);
+                        ctx.fireChannelActive();
                        connectionPromise.trySuccess(connection);
                    }
                }