Update to opentelemetry-configuration v1.1.0#8451
Conversation
Codecov Report❌ Patch coverage is ❌ Your patch check has failed because the patch coverage (49.26%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #8451 +/- ##
============================================
+ Coverage 78.54% 79.13% +0.59%
- Complexity 8468 8518 +50
============================================
Files 1008 1012 +4
Lines 28824 28979 +155
Branches 3569 3601 +32
============================================
+ Hits 22639 22932 +293
+ Misses 5342 5161 -181
- Partials 843 886 +43 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| /** Reflectively set the max export batch size for the {@link SdkMeterProviderBuilder}. */ | ||
| public static SdkMeterProviderBuilder setMaxExportBatchSize( | ||
| SdkMeterProviderBuilder sdkMeterProviderBuilder, int maxExportBatchSize) { | ||
| public static PeriodicMetricReaderBuilder setMaxExportBatchSize( |
There was a problem hiding this comment.
This was wrong. Had to fix to add support for the new declarative config property for configuring batch size.
|
|
||
| @Override | ||
| public String getName() { | ||
| return "event_to_span_event_bridge/development"; |
There was a problem hiding this comment.
Now that event_to_span_event_bridge is formally supported in declarative config, we can directly reference instance of using SPI.
…y-java into declarative-config-1.1
| * The attribute value. The type of value must match .type. Property is required and must be | ||
| * non-null. | ||
| * The attribute value. The type of value must match .type. Property must be present, but if null | ||
| * the entry is ignored. |
There was a problem hiding this comment.
With POJOs checked into VCS now, its cool /useful to see these diffs as we update to the new version of the schema
Release notes: https://github.com/open-telemetry/opentelemetry-configuration/releases/tag/v1.1.0