Implementing configurable aggregation cardinality limit and collector emitting all views instead of last view#3299
Closed
PradSenn wants to merge 6 commits into
Closed
Implementing configurable aggregation cardinality limit and collector emitting all views instead of last view#3299PradSenn wants to merge 6 commits into
PradSenn wants to merge 6 commits into
Codecov / codecov/project
succeeded
Jul 23, 2025 in 1s
89.57% (+0.06%) compared to 4ecafb6
View this Pull Request on Codecov
89.57% (+0.06%) compared to 4ecafb6
Details
Codecov Report
All modified and coverable lines are covered by tests ✅
Project coverage is 89.57%. Comparing base (
4ecafb6) to head (3af667c).
Report is 131 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #3299 +/- ##
==========================================
+ Coverage 89.52% 89.57% +0.06%
==========================================
Files 210 210
Lines 6526 6529 +3
==========================================
+ Hits 5842 5848 +6
+ Misses 684 681 -3 | Files with missing lines | Coverage Δ | |
|---|---|---|
| ...metry/sdk/metrics/aggregation/aggregation_config.h | 100.00% <100.00%> (ø) |
|
| sdk/include/opentelemetry/sdk/metrics/meter.h | 57.15% <ø> (ø) |
|
| ...telemetry/sdk/metrics/state/async_metric_storage.h | 94.60% <100.00%> (ø) |
|
| ...entelemetry/sdk/metrics/state/attributes_hashmap.h | 97.02% <100.00%> (+4.71%) |
⬆️ |
| ...ntelemetry/sdk/metrics/state/sync_metric_storage.h | 86.37% <100.00%> (ø) |
|
| sdk/src/metrics/meter.cc | 82.50% <100.00%> (ø) |
|
| sdk/src/metrics/state/sync_metric_storage.cc | 100.00% <100.00%> (ø) |
|
| sdk/src/metrics/state/temporal_metric_storage.cc | 97.37% <100.00%> (ø) |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Loading