1fanwang commented on PR #66810: URL: https://github.com/apache/airflow/pull/66810#issuecomment-4626733446
Addressed the scheduled/deferred question (yours and @ferruzzi's) — added both in fb84642, so all five slot states are covered now. On OTel: traced the path and I think you're right that it's out of scope. `timing()` records into a real histogram — `MetricsMap.record_histogram_value` → `InternalHistogram` calls `meter.create_histogram(name, unit="ms")` then `.record(value)`, and the names pass `name_is_otel_safe` (`OTEL_NAME_MAX_LENGTH` is 255; `airflow.pool.*_slots.distribution` is ~37 chars). So airflow does emit these as proper OTel histograms — the gap is collector/export-side, which matches your 0.70 → 0.85 observation and would hit any `stats.timing` metric, not just these. Nothing to change here for it. -- 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]
