shibd commented on a change in pull request #13184:
URL: https://github.com/apache/pulsar/pull/13184#discussion_r768358417
##########
File path:
pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/NamespacesImpl.java
##########
@@ -146,6 +147,56 @@ public void failed(Throwable throwable) {
return future;
}
+ @Override
+ public List<BundleStats> getAllBundleStats(String namespace) throws
PulsarAdminException {
+ return sync(() -> getAllBundleStatsAsync(namespace));
+ }
+
+ @Override
+ public CompletableFuture<List<BundleStats>> getAllBundleStatsAsync(String
namespace) {
+ NamespaceName ns = NamespaceName.get(namespace);
+ WebTarget path = namespacePath(ns, "bundleStats");
+ final CompletableFuture<List<BundleStats>> future = new
CompletableFuture<>();
+ asyncGetRequest(path,
Review comment:
Will there be a lookup here? All bundles of a namespace will be on
different brokers.
--
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]
