ableegoldman commented on a change in pull request #9688:
URL: https://github.com/apache/kafka/pull/9688#discussion_r538757741
##########
File path:
streams/src/test/java/org/apache/kafka/streams/integration/EosBetaUpgradeIntegrationTest.java
##########
@@ -540,18 +535,18 @@ public void shouldUpgradeFromEosAlphaToEosBeta() throws
Exception {
}
// 7. only for crash case:
- // 7a. restart the second client in eos-alpha mode and wait
until rebalance stabilizes
+ // 7a. restart the failed second client in eos-alpha mode and
wait until rebalance stabilizes
Review comment:
I think "failed second client" is correct. It's the 2nd client, which
has failed, not the 2nd client to have failed (English is confusing 😣 )
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
