[GitHub] [zookeeper] symat opened a new pull request #1176: ZOOKEEPER-1105: wait for server response in C client zookeeper_close

[GitBox <gi...@apache.org>]

symat opened a new pull request #1176: ZOOKEEPER-1105: wait for server response in C client zookeeper_close
URL: https://github.com/apache/zookeeper/pull/1176
 
 
   **Thanks for Lincoln Lee for the original fix!**
   
   In the current implementation, we always get a WARN in server side
   ("EndOfStreamException: Unable to read additional data from client")
   whenever we close a zookeeper handler from the C client. This also happens
   in the end of every execution of the command line C client.
   
   The reason is that currently we don't wait for the response from the server
   when we initiate the closing of the client connection, and we terminate
   the socket on the client side too early.
   
   I tested the patch both on linux and windows. I also tested it both with
   NIO and Netty server side socket implementations.

----------------------------------------------------------------
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