dao-jun commented on code in PR #22490:
URL: https://github.com/apache/pulsar/pull/22490#discussion_r1565469039


##########
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##########
@@ -2915,7 +2915,9 @@ The max allowed delay for delayed delivery (in 
milliseconds). If the broker rece
     private boolean exposeTopicLevelMetricsInPrometheus = true;
     @FieldContext(
             category = CATEGORY_METRICS,
-            doc = "If true, export buffered metrics"
+            doc = "Set to true to enable the broker to cache the metrics 
response, default is false. "
+                    + "For scraping metrics more than once per scrape period, 
the broker generates metrics at the first "
+                    + "scrape and returns the same response for the rest of 
the scrape period. "

Review Comment:
   fix, PTAL



##########
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##########
@@ -2915,7 +2915,9 @@ The max allowed delay for delayed delivery (in 
milliseconds). If the broker rece
     private boolean exposeTopicLevelMetricsInPrometheus = true;
     @FieldContext(
             category = CATEGORY_METRICS,
-            doc = "If true, export buffered metrics"
+            doc = "Set to true to enable the broker to cache the metrics 
response, default is false. "
+                    + "For scraping metrics more than once per scrape period, 
the broker generates metrics at the first "
+                    + "scrape and returns the same response for the rest of 
the scrape period. "

Review Comment:
   fixed, PTAL



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to