Cloudwatch metrics does not limit nr. of MetricDatums to 20 per call

[feliksik]

According to the AWS Docs, a max of 20 metricdatums can be sent in an API request.

So this code should respect that.

I might get to fixing this myself, but not today...

[cam-stitt]

Good find. Thanks for this.

[alpeb]

Would it suffice if I add checks for that in NewCounter, NewGauge and NewHistogram that would panic if that limit is reached?

[cam-stitt]

I'm not sure if a panic would be ideal in this situation. We should be able to split it into a max of 20 datums and spread it to multiple api requests.

[alpeb]

Sounds good. Can I have the honor of submitting a PR for this, or you prefer to tackle it yourself? :)
I was thinking we could do it in batches of concurrent API requests, say up to 4 concurrent requests per batch. Let me know what you think.

[cam-stitt]

Go for it! Happy to share. I would suggest making an option NumConcurrentRequests to specify how many requests can be sent per batch.

[cam-stitt]

@peterbourgon I believe we can probably close this off now?

[peterbourgon]

Jawohl.