This is an automated email from the ASF dual-hosted git repository.

lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new c24953035b8 [improve][broker]Reuse method getAvailableBrokersAsync 
(#23099)
c24953035b8 is described below

commit c24953035b88e670541c7f736514cd6b260e6520
Author: crossoverJie <[email protected]>
AuthorDate: Wed Jul 31 14:10:40 2024 +0800

    [improve][broker]Reuse method getAvailableBrokersAsync (#23099)
---
 .../apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java
 
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java
index ada1ab665b6..141e020d7ca 100644
--- 
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java
+++ 
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java
@@ -350,8 +350,7 @@ public class ModularLoadManagerImpl implements 
ModularLoadManager {
     @Override
     public Set<String> getAvailableBrokers() {
         try {
-            return new 
HashSet<>(brokersData.listLocks(LoadManager.LOADBALANCE_BROKERS_ROOT)
-                    .get(conf.getMetadataStoreOperationTimeoutSeconds(), 
TimeUnit.SECONDS));
+            return 
getAvailableBrokersAsync().get(conf.getMetadataStoreOperationTimeoutSeconds(), 
TimeUnit.SECONDS);
         } catch (Exception e) {
             log.warn("Error when trying to get active brokers", e);
             return loadData.getBrokerData().keySet();

Reply via email to