swamymavuri commented on code in PR #19487:
URL: https://github.com/apache/pulsar/pull/19487#discussion_r1115841310


##########
pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/TopicsImpl.java:
##########
@@ -199,20 +204,62 @@ public CompletableFuture<List<String>> 
getListAsync(String namespace, TopicDomai
                 .queryParam("includeSystemTopic", 
options.isIncludeSystemTopic());
         final CompletableFuture<List<String>> persistentList;
         final CompletableFuture<List<String>> nonPersistentList;
+        final CompletableFuture<List<String>> nonPersistentPartitionedList;

Review Comment:
   persistentList retrieves all the topics which includes partitioned and 
non-partitioned "persistent" topics(/persistent/tenant/namespace).
   getPersistentList retrives data from the path 
/managed-ledger-path/tenant/namespace/persistent which contains list of both 
partitioned and non-partitioned.
   But nonPersistentList retrieves topics only from the namespace owned bundle 
and rest of the bundle topics were skipped.
   That is the reason we are making non-persistent/tenant/namespace/partitioned 
call from the client to include non persistent partitioned topics.
   
   



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to