zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eolivelli <...@git.apache.org>
Subject [GitHub] zookeeper pull request #669: ZOOKEEPER-3152: Port ZK netty stack to netty4
Date Sat, 20 Oct 2018 18:40:32 GMT
Github user eolivelli commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/669#discussion_r226835980
  
    --- Diff: zookeeper-common/src/main/java/org/apache/zookeeper/ClientCnxnSocketNetty.java
---
    @@ -267,7 +298,7 @@ private void sendPkt(Packet p) {
             p.createBB();
             updateLastSend();
             sentCount++;
    -        channel.write(ChannelBuffers.wrappedBuffer(p.bb));
    +        channel.writeAndFlush(Unpooled.wrappedBuffer(p.bb));
    --- End diff --
    
    See these very interesting slides 
    http://normanmaurer.me/presentations/2014-facebook-eng-netty/slides.html#8.0


---

Mime
View raw message