yfxhust commented on issue #1235: ZOOKEEPER-3706: ZooKeeper.close() would leak SendThread when the netw… URL: https://github.com/apache/zookeeper/pull/1235#issuecomment-582822785 > left two more comments - current approach should work but I am wondering if my proposal would sound better in term of more structurally solve the race condition around the state variable; also was a little bit concerning around RuntimeException we throw and the implications it brought (w.r.t resource leaks) Thank you for your comments. I adapt my code with your suggestions. Please review the latest patch.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
