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/08/06 14:32:49 UTC
(pulsar) 03/08: [improve][broker]Reuse method getAvailableBrokersAsync (#23099)
This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/pulsar.git
commit e0fccdbe665ec15cd7c67494222f28028b82dddd
Author: crossoverJie <cr...@gmail.com>
AuthorDate: Wed Jul 31 14:10:40 2024 +0800
[improve][broker]Reuse method getAvailableBrokersAsync (#23099)
(cherry picked from commit c24953035b88e670541c7f736514cd6b260e6520)
---
.../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 ddcc3713ec0..735319c7995 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
@@ -353,8 +353,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();