ijuma commented on code in PR #14661:
URL: https://github.com/apache/kafka/pull/14661#discussion_r1375311183


##########
tools/src/main/java/org/apache/kafka/tools/TopicCommand.java:
##########
@@ -469,7 +469,7 @@ public void createTopic(CommandTopicPartition topic) throws 
Exception {
                 CreateTopicsResult createResult = 
adminClient.createTopics(Collections.singleton(newTopic),
                     new CreateTopicsOptions().retryOnQuotaViolation(false));
                 createResult.all().get();
-                System.out.println("Created topic " + topic.name + ".");
+                System.out.println("Created topic " + topic.name.get() + ".");

Review Comment:
   Thanks for the fix. I think we should pass the topic name to `createTopic` 
so we don't have to call `.get` in so many places.



-- 
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

Reply via email to