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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7c414c12b ZOOKEEPER-4951: Optimize the default Prometheus max queue 
size (#2278)
7c414c12b is described below

commit 7c414c12b5c59e64aa9c52277396f4c42032ca48
Author: li4wang <[email protected]>
AuthorDate: Thu Sep 4 10:52:51 2025 -0700

    ZOOKEEPER-4951: Optimize the default Prometheus max queue size (#2278)
    
    Author: Li Wang <[email protected]>
    
    Reviewer: kezhuw <[email protected]>
---
 zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md        | 2 +-
 .../zookeeper/metrics/prometheus/PrometheusMetricsProvider.java     | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md 
b/zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md
index 5bac829cf..b14cb6b86 100644
--- a/zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md
+++ b/zookeeper-docs/src/main/resources/markdown/zookeeperAdmin.md
@@ -2279,7 +2279,7 @@ options are used to configure the 
[AdminServer](#sc_adminserver).
 * *metricsProvider.maxQueueSize* :
    **New in 3.7.1:**
    The max queue size for Prometheus summary metrics reporting task.
-   Default value is 1000000.
+   Default value is 10000.
    
 * *metricsProvider.workerShutdownTimeoutMs* :
    **New in 3.7.1:**
diff --git 
a/zookeeper-metrics-providers/zookeeper-prometheus-metrics/src/main/java/org/apache/zookeeper/metrics/prometheus/PrometheusMetricsProvider.java
 
b/zookeeper-metrics-providers/zookeeper-prometheus-metrics/src/main/java/org/apache/zookeeper/metrics/prometheus/PrometheusMetricsProvider.java
index d41b0ad5a..3745a753c 100644
--- 
a/zookeeper-metrics-providers/zookeeper-prometheus-metrics/src/main/java/org/apache/zookeeper/metrics/prometheus/PrometheusMetricsProvider.java
+++ 
b/zookeeper-metrics-providers/zookeeper-prometheus-metrics/src/main/java/org/apache/zookeeper/metrics/prometheus/PrometheusMetricsProvider.java
@@ -84,7 +84,7 @@ public class PrometheusMetricsProvider implements 
MetricsProvider {
 
     /**
      * The max queue size for Prometheus summary metrics reporting task.
-     * Default value is 1000000.
+     * Default value is 10000.
      */
     static final String MAX_QUEUE_SIZE = "maxQueueSize";
 
@@ -111,7 +111,7 @@ public class PrometheusMetricsProvider implements 
MetricsProvider {
     private final MetricsServletImpl servlet = new MetricsServletImpl();
     private final Context rootContext = new Context();
     private int numWorkerThreads = 1;
-    private int maxQueueSize = 1000000;
+    private int maxQueueSize = 10000;
     private long workerShutdownTimeoutMs = 1000;
     private Optional<ExecutorService> executorOptional = Optional.empty();
 
@@ -161,7 +161,7 @@ public void configure(Properties configuration) throws 
MetricsProviderLifeCycleE
         this.numWorkerThreads = Integer.parseInt(
                 configuration.getProperty(NUM_WORKER_THREADS, "1"));
         this.maxQueueSize = Integer.parseInt(
-                configuration.getProperty(MAX_QUEUE_SIZE, "1000000"));
+                configuration.getProperty(MAX_QUEUE_SIZE, "10000"));
         this.workerShutdownTimeoutMs = Long.parseLong(
                 configuration.getProperty(WORKER_SHUTDOWN_TIMEOUT_MS, "1000"));
     }

Reply via email to