You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2023/12/04 22:50:44 UTC

(pinot) branch master updated: Fix default brokerUpdateFrequencyInMillis for connector (#12093)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 9497f06cba Fix default brokerUpdateFrequencyInMillis for connector (#12093)
9497f06cba is described below

commit 9497f06cbab4817d78714d2fc19b3e67eaa18ee0
Author: Xiaotian (Jackie) Jiang <17...@users.noreply.github.com>
AuthorDate: Mon Dec 4 14:50:38 2023 -0800

    Fix default brokerUpdateFrequencyInMillis for connector (#12093)
---
 .../java/org/apache/pinot/client/BrokerCacheUpdaterPeriodic.java  | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerCacheUpdaterPeriodic.java b/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerCacheUpdaterPeriodic.java
index 3c4080bac2..c8d8b58049 100644
--- a/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerCacheUpdaterPeriodic.java
+++ b/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerCacheUpdaterPeriodic.java
@@ -31,17 +31,15 @@ import org.slf4j.LoggerFactory;
  */
 public class BrokerCacheUpdaterPeriodic implements UpdatableBrokerCache {
   public static final String BROKER_UPDATE_FREQUENCY_MILLIS = "brokerUpdateFrequencyInMillis";
-  public static final String DEFAULT_BROKER_UPDATE_FREQUENCY_MILLIS = "defaultBrokerUpdateFrequencyInMillis";
+  public static final String DEFAULT_BROKER_UPDATE_FREQUENCY_MILLIS = "300000"; // 5 minutes
+
+  private static final Logger LOGGER = LoggerFactory.getLogger(BrokerCacheUpdaterPeriodic.class);
 
   private final BrokerCache _brokerCache;
   private final ScheduledExecutorService _scheduledExecutorService;
   private final long _brokerUpdateFreqInMillis;
-  private final Properties _properties;
-
-  private static final Logger LOGGER = LoggerFactory.getLogger(BrokerCacheUpdaterPeriodic.class);
 
   public BrokerCacheUpdaterPeriodic(Properties properties, String controllerUrl) {
-    _properties = properties;
     _brokerCache = new BrokerCache(properties, controllerUrl);
     _scheduledExecutorService = Executors.newSingleThreadScheduledExecutor();
     _brokerUpdateFreqInMillis = Long.parseLong(


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org