Skip to content
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

Provide an example for using ErrorType metrics #5067

Open
yattoni opened this issue Apr 4, 2024 · 0 comments
Open

Provide an example for using ErrorType metrics #5067

yattoni opened this issue Apr 4, 2024 · 0 comments
Labels
documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged.

Comments

@yattoni
Copy link

yattoni commented Apr 4, 2024

Describe the issue

In your design for metrics and in the source code there exists an ErrorType metric. However it is a String which cannot be turned into a metric in the TimeBucketedMetrics class.

Since it is a String type, I believe it can only be used for Dimensions, which would mean I would have to extend the defaults to see it. But also, what metric would I use to alarm on if I had metrics with the error type dimension set? ApiCallSuccessful? But it wouldn't be successful so I'm just looking for the sample count of that?

The ErrorType also only gets recorded on exceptions here so unless the AWS service is failing I will never see ServerError metrics to create alarms for or know that I am graphing correctly. Ideally these metrics get collected with zeros on every call.

Ideally I think the ServerError and Throttling just become their own standard metrics which teams can create alarms on and it would also simplify the current approach of doing SampleCount - Sum for ApiCallSuccessful, which was the workaround proposed here #2711. Should also add ClientError to the list of possibilities.

ErrorType is not mentioned in metrics here https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/metrics-list.html implying it is a dimension, but again, those metrics won't exist unless the API's are failing.

Links

https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/metrics-list.html

@yattoni yattoni added documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged. labels Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

1 participant