You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "amaechler (via GitHub)" <gi...@apache.org> on 2026/08/22 03:22:47 UTC
[PR] Emit `taskStatus` on `task/run/time` from the `statsd` and `prometheus` emitters (druid)
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
Re: [PR] minor: Emit `taskStatus` on `task/run/time` from the `statsd` and `prometheus` emitters (druid)
Posted by "FrankChen021 (via GitHub)" <gi...@apache.org>.
FrankChen021 merged PR #20122:
URL: https://github.com/apache/druid/pull/20122
--
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-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
Re: [PR] minor: Emit `taskStatus` on `task/run/time` from the `statsd` and `prometheus` emitters (druid)
Posted by "kfaraz (via GitHub)" <gi...@apache.org>.
kfaraz commented on PR #20122:
URL: https://github.com/apache/druid/pull/20122#issuecomment-5391021935
@amaechler , could you please resolve the merge conflicts?
--
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-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org