rajinisivaram commented on a change in pull request #9622:
URL: https://github.com/apache/kafka/pull/9622#discussion_r544977551
##########
File path: clients/src/main/java/org/apache/kafka/common/Cluster.java
##########
@@ -327,6 +346,14 @@ public Node controller() {
return controller;
}
+ public Collection<Uuid> topicIds() {
+ return topicIds.values();
+ }
+
+ public Uuid getTopicId(String topic) {
Review comment:
nit: We don't use `get` prefix in other methods. Since this is part of
the public API, can we change this to `topicId(String topic)` before merging?
----------------------------------------------------------------
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
