[GitHub] [pulsar] aloyszhang commented on a change in pull request #12041: make partitioned-lookup result more readable

[GitBox <gi...@apache.org>]

aloyszhang commented on a change in pull request #12041:
URL: https://github.com/apache/pulsar/pull/12041#discussion_r713841941



##########
File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Lookup.java
##########
@@ -57,7 +58,7 @@
      * @param topic
      * @return the broker URLs that serves the topic
      */
-    CompletableFuture<Map<String, String>> lookupPartitionedTopicAsync(String topic);
+    CompletableFuture<Map<String, List<String>>> lookupPartitionedTopicAsync(String topic);

Review comment:
       Sorry for the misunderstanding. I have modified the API implmention. 
   The API will keep compatibility, `patitioned-lookup` result is still the type of `Map<String, String>`.
   For the new display, add an option `--key-by-broker-url` which default is false.
   If enalbe `--key-by-broker-url`, the result will be keyed by broekrUrl, and the value is the `List<String>.toString` which contains all partitions belong to the topic on this broker.




-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org