[Metrics SDK] Implement configurable cardinality limit#4188
Open
om7057 wants to merge 6 commits into
Open
Codecov / codecov/project
succeeded
Jul 1, 2026 in 1s
82.92% (+0.04%) compared to a56f0e2
View this Pull Request on Codecov
82.92% (+0.04%) compared to a56f0e2
Details
Codecov Report
❌ Patch coverage is 90.69767% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.92%. Comparing base (a56f0e2) to head (2895f04).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| ...opentelemetry/sdk/metrics/state/metric_collector.h | 0.00% | 2 Missing |
| sdk/src/metrics/metric_reader.cc | 93.75% | 2 Missing |
Additional details and impacted files
@@ Coverage Diff @@
## main #4188 +/- ##
==========================================
+ Coverage 82.89% 82.92% +0.04%
==========================================
Files 405 405
Lines 17292 17335 +43
==========================================
+ Hits 14332 14373 +41
- Misses 2960 2962 +2 | Files with missing lines | Coverage Δ | |
|---|---|---|
| .../include/opentelemetry/sdk/metrics/metric_reader.h | 66.67% <ø> (ø) |
|
| sdk/src/metrics/meter.cc | 81.36% <ø> (ø) |
|
| sdk/src/metrics/meter_context.cc | 87.78% <100.00%> (+1.04%) |
⬆️ |
| sdk/src/metrics/state/metric_collector.cc | 96.78% <100.00%> (+0.11%) |
⬆️ |
| ...opentelemetry/sdk/metrics/state/metric_collector.h | 60.00% <0.00%> (-40.00%) |
⬇️ |
| sdk/src/metrics/metric_reader.cc | 87.70% <93.75%> (+5.88%) |
⬆️ |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Loading