You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by lh...@apache.org on 2024/07/31 06:10:46 UTC
(pulsar) branch master updated: [improve][broker]Reuse method getAvailableBrokersAsync (#23099)
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 <cr...@gmail.com>
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();