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();