@sijie I think superficially it can look like it takes longer, but in practice it should only ever take less time.
The dominating latency in this operation is the write to zookeeper(LatWrite). In the case where we do not block other changes to the ensemble, and there are two failures, one of the updates will fail, have to reread (LatRead) and write again. So the latency is (LatWrite + LatRead + LatWrite). With the new code, it's just LatWrite + LatWrite. It gets better if there's more than 2 failures. [ Full content available at: https://github.com/apache/bookkeeper/pull/1646 ] This message was relayed via gitbox.apache.org for [email protected]
