I don't think any synchronization is needed here either. newEnsemblesFromRecovery will only ever be greater than 0 during the recovery process, and during the recovery process, getCurrentEnsemble is only called in a callback (which run in the handle's executor, same as handleBookieFailure. That said, the whole threading model is a mess, so I'll add defensive synchronization until we clear it up.
[ Full content available at: https://github.com/apache/bookkeeper/pull/1621 ] This message was relayed via gitbox.apache.org for [email protected]
