IcebreakerSA opened a new pull request, #72713: URL: https://github.com/apache/airflow/pull/72713
Fixes the 1000x under-reporting of timer metrics when the DogStatsD backend is enabled. `Stats.timer()` measures durations in milliseconds, but `datadog.DogStatsd.timed()` defaults to seconds unless `use_ms` is enabled. This change enables millisecond output in the shared DogStatsD logger and adds a regression test for the constructor configuration. closes: #72110 Tests: - `uvx --offline ruff format shared/observability/src/airflow_shared/observability/metrics/datadog_logger.py shared/observability/tests/observability/metrics/test_stats.py` - `uvx --offline ruff check --fix shared/observability/src/airflow_shared/observability/metrics/datadog_logger.py shared/observability/tests/observability/metrics/test_stats.py` - `git diff --check` - Targeted Breeze test attempted but could not resolve the uncached `uv` dependency while offline; the online dependency resolution produced no output and was stopped. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Codex (GPT-5) Generated-by: Codex (GPT-5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
