Is it possible for isClosed() but not closing here? If closing, drainPendingAddsToErrorOut must have already happened (and no new ones can be added due to the check in doAsyncAddEntry), so this must necessarily be a noop. I think we should be able to check closing instead and assert that pendingAddOps is already empty.
[ Full content available at: https://github.com/apache/bookkeeper/pull/1646 ] This message was relayed via gitbox.apache.org for [email protected]
