capistrant commented on code in PR #17689:
URL: https://github.com/apache/druid/pull/17689#discussion_r1940251487
##########
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:
ya, I'm ok with us leaving it to the operator to decide if/when they want to
take advantage of this change
--
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]
