This discussion becomes moot with how I've changed it now. We do call drain() in all cases, but I've done it this way to only have one synchronized block. In the case that the ledger is already closed, the drain should just return an empty list.
[ Full content available at: https://github.com/apache/bookkeeper/pull/1646 ] This message was relayed via gitbox.apache.org for [email protected]
