1fanwang opened a new issue, #66801:
URL: https://github.com/apache/airflow/issues/66801

   ### Description
   
   PR #64207 (merged) converts the OTel timer aggregation to 
`ExponentialBucketHistogramAggregation`, fixing the "timer becomes 
last-write-wins Gauge" bug. Good for timers.
   
   For non-timer metrics — counts, durations on direct `Stats.histogram(...)`, 
custom `*_delay` metrics — there's no central place to declare 
per-metric-pattern bucket views. Each metric name family might want a different 
bucket shape (exponential for latencies, custom-bounded for counts), but the 
configuration lives at each call site instead of being declared once.
   
   ### Use case / motivation
   
   Standardize bucket configuration across the metric vocabulary so operators 
don't have to chase per-call defaults to interpret a histogram.
   
   ### Proposal
   
   Sketch: a small `airflow/metrics/histogram_buckets.py` module that registers 
OTel `View` objects keyed by metric-name pattern (`*_count`, `*_duration`, 
`*_delay`, ...). Default patterns ship with sensible buckets; deployments can 
register additional patterns.
   
   I'm flagging this as a design discussion rather than a ready PR — there's a 
reasonable counter-argument that "let each call site pass its bucket 
boundaries" is the right shape and a central module is over-engineering. Filing 
here so the discussion has a permanent reference.
   
   ### Are you willing to submit a PR?
   
   - [X] Yes, after design feedback
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's Code of Conduct
   


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

Reply via email to