-
Notifications
You must be signed in to change notification settings - Fork 104
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
Add Batch Limit for Stackdriver Exporter #575
Comments
Thanks for reporting this! This is definitely something we need to implement. It exists on the googlecloud collector exporter for metrics, but not on the SDK exporter right now. |
Would it be possible to get this ticket prioritized? I currently can't use the latest version of the exporter in production because it will constantly throw the maximum time series error. Thanks! |
@dashpole I can work on this if you'd like |
@damemi go for it. |
hi @damemi was there any progress on this? |
A naive implementation could be main...knightpp:opentelemetry-operations-go:issue-575. |
@zchee I've created a PR. I hope the PR is good enough to be merged. Thanks. |
On high-velocity metrics, you get:
Apparently Stackdriver backend has the limit that each request cannot batch more than 200 time series so exporter needs to do some processing.
I have this issue with f0e0c76 commit
See GoogleCloudPlatform/opentelemetry-operations-js/issues/87
The text was updated successfully, but these errors were encountered: