merlimat commented on PR #23011:
URL: https://github.com/apache/pulsar/pull/23011#issuecomment-2214206694
@poorbarcode The behavior is intended to prevent a temporary loss of a ZK
session, to bring down an entire cluster of brokers at the same time.
In the example above, when broker-1 will be able to re-establish a session
with ZK, it will get the notification that the ownership has changed.
For the semantics guarantees, there is no problem or race condition:
* The ledgers where broker-1 was writing are getting fenced by the new
broker that takes over
* broker-1 is not able to create a new ledger (because of no zk session)
and won't anyway be able to update the ledger list, since the version has
already changed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
