Demogorgon314 commented on code in PR #20058:
URL: https://github.com/apache/pulsar/pull/20058#discussion_r1162184289
##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/extensions/ExtensibleLoadManagerImpl.java:
##########
@@ -106,6 +107,8 @@ public class ExtensibleLoadManagerImpl implements
ExtensibleLoadManager {
private static final long MONITOR_INTERVAL_IN_MILLIS = 120_000;
+ private static final String ELECTION_ROOT =
"/loadbalance/extension/leader";
Review Comment:
If the same leader plays both old and new LM leader, it will have problems.
When lookup the LM system topic, it might redirect to the old leader broker,
but it doesn't start the new LM in this broker, so it will redirect to the new
LM broker again...
--
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]
