ivankelly commented on issue #1132: Fix shutdown race which left ZK session open
URL: https://github.com/apache/bookkeeper/pull/1132#issuecomment-364877305
   @sijie yes, that change would also fix the immediate issue. It could also be 
fixed in a number of one-line ways.
   - synchronization on #close() method
   - Simply removing the #close() in #main()
   I can pare down the patch to this if needed, but right now it's trying to 
address some fundamental issues with the lifecycle stuff, i.e. that it's taking 
action based on perceived state, and updating that state, without any 
synchronization between the threads.

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

With regards,
Apache Git Services

Reply via email to