amaechler opened a new pull request, #20122:
URL: https://github.com/apache/druid/pull/20122

   This PR adds `taskStatus` to `task/run/time` in the Prometheus emitter, 
which will allow to easily group run times by success status (and alert on 
outliers).
   
   ### Scope
   
   `taskStatus` takes at most two values here (`SUCCESS`, `FAILED`), so this at 
most doubles the cardinality.
   
   #### Release note
   
   The statsd and prometheus emitters now emit the `taskStatus` dimension on 
`task/run/time` This makes it possible to count task successes and failures per 
task type through these emitters.
   
   This is a breaking change for existing dashboards and alerts on 
`task/run/time`:
   
   - **statsd (non-dogstatsd)**: dimension values are joined into the dotted 
metric name, so `druid.overlord.task.run.time.<dataSource>.<taskType>` becomes 
`druid.overlord.task.run.time.<dataSource>.<taskStatus>.<taskType>`. Queries 
against the old name stop receiving data.
   - **statsd (dogstatsd)**: a `taskStatus` tag is added; existing queries keep 
working, aggregating over it.
   - **prometheus**: a `taskStatus` label is added. Queries selecting on an 
exact label set, and recording rules assuming the old set, need updating.
   
   Operators wanting the previous behaviour can pin the old dimension list with 
`druid.emitter.statsd.dimensionMapPath` or 
`druid.emitter.prometheus.dimensionMapPath`.
   


-- 
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]

Reply via email to