poorbarcode opened a new pull request, #17861: URL: https://github.com/apache/pulsar/pull/17861
Fixes: - https://github.com/apache/pulsar/issues/16980 ### Motivation This test uses `setup()` and `PulsarService pulsar2 = startBroker(conf2)` to create two brokers, but the both brokers use the same `zkc`, so both brokers will share the `session` of `ZK`, which will lead to the possibility that the two brokers will constantly elect for the leader. see code: https://github.com/apache/pulsar/blob/b89c1451551a6bbe681465726906a2e61c9d8a69/pulsar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/LeaderElectionImpl.java#L143-L147 ### Modifications Make two brokers use different `session` ### Documentation - [x] `doc-not-needed` ### Matching PR in forked repository PR in forked repository: - https://github.com/poorbarcode/pulsar/pull/15 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
