oneby-wang commented on code in PR #24762:
URL: https://github.com/apache/pulsar/pull/24762#discussion_r2362649408
##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java:
##########
@@ -2907,5 +2908,16 @@ protected CompletableFuture<Set<String>>
internalGetNamespaceAllowedClustersAsyn
.thenApply(policies -> policies.allowed_clusters);
}
-
+ private BundlesData getDefaultBundleData() {
+ Optional<Policies> nsPolicies;
+ try {
+ nsPolicies = namespaceResources().getPolicies(namespaceName);
Review Comment:
There are many synchronous calls in the admin v2 Namespaces. I had no choice
but to use a synchronous call here because the upstream methods are using
synchronous calls. I plan to submit several PRs to replace the synchronous
operations with asynchronous ones after this PR in admin v2 Namespaces. WDYT?
--
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]
