chia7712 commented on code in PR #15946: URL: https://github.com/apache/kafka/pull/15946#discussion_r1600223978
########## core/src/test/java/kafka/test/junit/RaftClusterInvocationContext.java: ########## @@ -284,24 +258,44 @@ public void startBroker(int brokerId) { @Override public void waitForReadyBrokers() throws InterruptedException { try { - clusterReference.get().waitForReadyBrokers(); + clusterTestKit.waitForReadyBrokers(); } catch (ExecutionException e) { throw new AssertionError("Failed while waiting for brokers to become ready", e); } } - private BrokerServer findBrokerOrThrow(int brokerId) { - return Optional.ofNullable(clusterReference.get().brokers().get(brokerId)) - .orElseThrow(() -> new IllegalArgumentException("Unknown brokerId " + brokerId)); - } - public Stream<BrokerServer> brokers() { - return clusterReference.get().brokers().values().stream(); + return clusterTestKit.brokers().values().stream(); } public Stream<ControllerServer> controllers() { - return clusterReference.get().controllers().values().stream(); + return clusterTestKit.controllers().values().stream(); } + private BrokerServer findBrokerOrThrow(int brokerId) { + return Optional.ofNullable(clusterTestKit.brokers().get(brokerId)) + .orElseThrow(() -> new IllegalArgumentException("Unknown brokerId " + brokerId)); + } + + private void buildAndFormatCluster() throws Exception { Review Comment: Could we add a phase `format` for kraft type? There are some test cases requiring `format` -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org