feat(sdk-metrics)!: extract IMetricReader interface and use it over abstract class #5311
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Which problem is this PR solving?
We periodically run into problems since private properties in classes cause the type to be incompatible between versions. This PR fixes that for
MetricReader
by extracting an interface which we then use instead of the abstract class type in our public interface.In this PR
MetricReader
type stays as-is, but this is marked as a breaking change since we technically narrow down what's being provided byMeterProviderOptions
.Part of #5283
Type of change