zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anmolnar <...@git.apache.org>
Subject [GitHub] zookeeper pull request #669: ZOOKEEPER-3152: Port ZK netty stack to netty4
Date Wed, 14 Nov 2018 23:22:50 GMT
Github user anmolnar commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/669#discussion_r233659921
  
    --- Diff: zookeeper-server/src/main/java/org/apache/zookeeper/server/NettyServerCnxnFactory.java
---
    @@ -316,16 +251,17 @@ public void operationComplete(ChannelFuture future)
                         if (KeeperException.Code.OK !=
                                 authProvider.handleAuthentication(cnxn, null)) {
                             LOG.error("Authentication failed for session 0x{}",
    -                                Long.toHexString(cnxn.sessionId));
    +                                Long.toHexString(cnxn.getSessionId()));
                             cnxn.close();
                             return;
                         }
     
    -                    allChannels.add(future.getChannel());
    +                    final Channel futureChannel = future.getNow();
    --- End diff --
    
    It's fine.


---

Mime
View raw message