Luke Chen
Luke Chen
> As mentioned above if we are on the same page then creating a JIRA issue to do additional checking to see if hosts are valid seems appropriate. Yes, please...
@mimaison @tombentley , do you want to have another look?
I'll review it this week. Sorry for the delay.
Triggering another jenkins build to make sure it passes the tests. https://ci-builds.apache.org/job/Kafka/job/kafka-pr/view/change-requests/job/PR-12500/2/
> Ah, sorry for the delay. I think time.sleep() here is needed here to make the pollTimer timeout. If we removed this, will hang in the coordinator.onJoinPrepare @aiquestion , thanks...
Weird! Looks like the test still stuck. I'll take a look after my vacation.
@jsancio , I think we should make this fix into v3.3, otherwise, the metadata shell is basically broken. WDYT? @dengziming , could you help confirm if this patch contain all...
Thank you! Let's wait for the jenkins build. @jsancio , I'm going to backport this patch to 3.3 tomorrow if you have no objection. Thanks.
Failed tests are un-related: ``` Build / JDK 11 and Scala 2.13 / kafka.server.KRaftClusterTest.testCreateClusterAndPerformReassignment() ```
backport to 3.3