Skip to content

[SDK] fix: fixed size exemplar reservoir works correctly - #4429

Open
proost wants to merge 11 commits into
open-telemetry:mainfrom
proost:fix-fixed-size-exemplar-reservoir
Open

[SDK] fix: fixed size exemplar reservoir works correctly#4429
proost wants to merge 11 commits into
open-telemetry:mainfrom
proost:fix-fixed-size-exemplar-reservoir

Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…

dcff7a2
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 29, 2026 in 0s

83.30% (+0.09%) compared to d3b1670

View this Pull Request on Codecov

83.30% (+0.09%) compared to d3b1670

Details

Codecov Report

❌ Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.30%. Comparing base (d3b1670) to head (dcff7a2).

Files with missing lines Patch % Lines
...cs/exemplar/simple_fixed_size_exemplar_reservoir.h 85.72% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4429      +/-   ##
==========================================
+ Coverage   83.22%   83.30%   +0.09%     
==========================================
  Files         520      520              
  Lines       20396    20404       +8     
==========================================
+ Hits        16972    16995      +23     
+ Misses       3424     3409      -15     
Files with missing lines Coverage Δ
...k/metrics/exemplar/fixed_size_exemplar_reservoir.h 91.18% <100.00%> (+1.53%) ⬆️
...cs/exemplar/simple_fixed_size_exemplar_reservoir.h 95.24% <85.72%> (+61.91%) ⬆️

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