Skip to content

Improve concurrent performance of exponential histogram measurements - #7535

Closed
dashpole wants to merge 1 commit into
open-telemetry:mainfrom
dashpole:optimize_exphist_ringbuffer
Closed

Improve concurrent performance of exponential histogram measurements#7535
dashpole wants to merge 1 commit into
open-telemetry:mainfrom
dashpole:optimize_exphist_ringbuffer

use sync.Map and atomics for exponential histogram

8871f03
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Dec 11, 2025 in 1s

86.1% (-0.1%) compared to f57bf14

View this Pull Request on Codecov

86.1% (-0.1%) compared to f57bf14

Details

Codecov Report

❌ Patch coverage is 87.86127% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.1%. Comparing base (f57bf14) to head (8871f03).

Files with missing lines Patch % Lines
sdk/metric/internal/aggregate/atomic.go 23.3% 23 Missing ⚠️
...metric/internal/aggregate/exponential_histogram.go 93.9% 11 Missing and 8 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #7535     +/-   ##
=======================================
- Coverage   86.2%   86.1%   -0.1%     
=======================================
  Files        298     298             
  Lines      21829   22003    +174     
=======================================
+ Hits       18817   18957    +140     
- Misses      2635    2664     +29     
- Partials     377     382      +5     
Files with missing lines Coverage Δ
sdk/metric/internal/aggregate/aggregate.go 100.0% <100.0%> (ø)
...metric/internal/aggregate/exponential_histogram.go 94.8% <93.9%> (-2.6%) ⬇️
sdk/metric/internal/aggregate/atomic.go 75.7% <23.3%> (-12.5%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.