[GitHub] [kafka] hachikuji commented on a change in pull request #11191: KAFKA-13173 Making the controller fence stale brokers one at a time if multiple stale brokers are discovered at the same time

2021-08-09 Thread GitBox
hachikuji commented on a change in pull request #11191: URL: https://github.com/apache/kafka/pull/11191#discussion_r685421603 ## File path: metadata/src/main/java/org/apache/kafka/controller/QuorumController.java ## @@ -858,7 +858,7 @@ private void rescheduleMaybeFenceStaleBro

[GitHub] [kafka] hachikuji commented on a change in pull request #11191: KAFKA-13173 Making the controller fence stale brokers one at a time if multiple stale brokers are discovered at the same time

2021-08-09 Thread GitBox
hachikuji commented on a change in pull request #11191: URL: https://github.com/apache/kafka/pull/11191#discussion_r685463058 ## File path: metadata/src/main/java/org/apache/kafka/controller/QuorumController.java ## @@ -858,7 +858,7 @@ private void rescheduleMaybeFenceStaleBro

[GitHub] [kafka] hachikuji commented on a change in pull request #11191: KAFKA-13173 Making the controller fence stale brokers one at a time if multiple stale brokers are discovered at the same time

2021-08-10 Thread GitBox
hachikuji commented on a change in pull request #11191: URL: https://github.com/apache/kafka/pull/11191#discussion_r686135001 ## File path: metadata/src/main/java/org/apache/kafka/controller/BrokerHeartbeatManager.java ## @@ -419,6 +419,27 @@ long nextCheckTimeNs() {

[GitHub] [kafka] hachikuji commented on a change in pull request #11191: KAFKA-13173 Making the controller fence stale brokers one at a time if multiple stale brokers are discovered at the same time

2021-08-11 Thread GitBox
hachikuji commented on a change in pull request #11191: URL: https://github.com/apache/kafka/pull/11191#discussion_r687186647 ## File path: metadata/src/test/java/org/apache/kafka/controller/QuorumControllerTest.java ## @@ -173,6 +176,121 @@ private void testDelayedConfigurat

[GitHub] [kafka] hachikuji commented on a change in pull request #11191: KAFKA-13173 Making the controller fence stale brokers one at a time if multiple stale brokers are discovered at the same time

2021-08-11 Thread GitBox
hachikuji commented on a change in pull request #11191: URL: https://github.com/apache/kafka/pull/11191#discussion_r687194150 ## File path: metadata/src/test/java/org/apache/kafka/controller/QuorumControllerTest.java ## @@ -173,6 +176,121 @@ private void testDelayedConfigurat