-
Notifications
You must be signed in to change notification settings - Fork 888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider removing Async Instruments (Observers) from first Metric version #1433
Comments
I could agree that for the first version this might be a bit difficult specifically on the SDK side, and while it would definitely be handy to have such primitives in the SDK side of it will likely take some time to experiment more than the synchronous side. There are plenty of edge cases especially with how an async sum would work given:
|
Doing async instruments properly seems rather complicated to me (e.g. #1432). If we specify synchronous instruments (especially BatchRecorder) properly, users should be able to implement a functionally equivalent solutions on their own (e.g. with their own background thread). (The only extra thing that might be needed for that is some "last-chance to collect" callback before every export.)
The text was updated successfully, but these errors were encountered: