zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] lvfangmin opened a new pull request #919: ZOOKEEPER-3356: Implement advanced Netty flow control based on feedback from ZK
Date Wed, 24 Apr 2019 23:46:39 GMT
lvfangmin opened a new pull request #919: ZOOKEEPER-3356: Implement advanced Netty flow control
based on feedback from ZK
URL: https://github.com/apache/zookeeper/pull/919
 
 
   The current implementation of enable/disable recv logic may cause the direct buffer OOM
because we may enable read a large chunk and disabled again after consume a single ZK request.
   
   This implementation disabled AUTO_READ and controls the READ depends on whether the SslHandler
has issued a READ and what's the queuedBuffer status.
   
   With this implementation, the max Netty queued buffer size (direct memory usage) will be
2 * recv_buffer size. It's not the per message size because in EPoll ET mode it will try to
read until the socket is empty, and because of SslHandler will trigger another read when it's
not a full encrypt packet and haven't issued any decrypt message.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message