You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Aggregator.diff is used to support delta export of asynchronous instruments, which are assumed to be cumulative. If we remove this method, than you can only export asynchronous instruments as cumualtive.
The SDK should not do this because may not be able to do it for all the Aggregators so if any of exporter needs this they can implement it. See open-telemetry/opentelemetry-go#2350
True. Right now Aggregator is an internal class because you can't define your own Aggregation. So I think its fine now, but something needed if we want to make it possible to define custom Aggregations.
Closing because: 1. Cumulative -> delta is necessary in the SDK for delta export of async counter and async up down counter. 2. Aggregator is an internal implementation detail.
No description provided.
The text was updated successfully, but these errors were encountered: