dajac commented on code in PR #12181:
URL: https://github.com/apache/kafka/pull/12181#discussion_r880753092
##########
metadata/src/main/java/org/apache/kafka/controller/BrokerHeartbeatManager.java:
##########
@@ -498,6 +498,23 @@ public UsableBroker next() {
}
}
+ /**
+ * Check if the given broker is active. A broker is active if it is
unfenced,
+ * and not shutting down.
Review Comment:
Yeah, that seems to be a reasonable tradeoff for 2). We could also improve
this later on if it turns out the be an issue. I believe that receiving the
FencedBroker record will be faster for 2) than for 1) in general.
--
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]
