ashwintumma23 commented on code in PR #17689:
URL: https://github.com/apache/druid/pull/17689#discussion_r1940256108
##########
docs/development/extensions-contrib/prometheus.md:
##########
@@ -40,7 +40,7 @@ All the configuration parameters for the Prometheus emitter
are under `druid.emi
| `druid.emitter.prometheus.strategy` | The strategy to expose
prometheus metrics. <br/>Should be one of `exporter` and `pushgateway`. Default
strategy `exporter` would expose metrics for scraping purpose. Peon tasks
(short-lived jobs) should use `pushgateway` strategy. | yes | exporter
|
| `druid.emitter.prometheus.port` | The port on which to expose
the prometheus HTTPServer. Required if using `exporter` strategy.
| no | none
|
| `druid.emitter.prometheus.namespace` | Optional metric namespace.
Must match the regex `[a-zA-Z_:][a-zA-Z0-9_:]*`
| no | druid
|
-| `druid.emitter.prometheus.dimensionMapPath` | JSON file defining the
Prometheus metric type, desired dimensions, help text, and conversionFactor for
every Druid metric.
| no | Default mapping
provided. See below. |
+| `druid.emitter.prometheus.dimensionMapPath` | JSON file defining the
Prometheus metric type, desired dimensions, conversionFactor, histogram buckets
and help text for every Druid metric.
| no |
Default mapping provided. See below. |
Review Comment:
Sounds good! Referencing quote/ comment below.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
