hanm commented on issue #1235: ZOOKEEPER-3706: ZooKeeper.close() would leak SendThread when the netw… URL: https://github.com/apache/zookeeper/pull/1235#issuecomment-582243797 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)
---------------------------------------------------------------- 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
