-
Notifications
You must be signed in to change notification settings - Fork 992
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
Service level objectives support on @Timed
annotation
#5145
Service level objectives support on @Timed
annotation
#5145
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the pull request. Would you please add test cases showing the Timer created from the TimedAspect and Timer builder method have the SLOs given in the annotation?
Could you also look into the TimedAspectTest failures? |
@shakuzen Thanks for taking a look, will do as soon as I can |
9b5ffa5
to
1485ba1
Compare
@Timed
annotation
The "enhancement" label seems to be missing for this, and an entry for this is also missing from the release notes for 1.14.0-M1. |
Thank you, I fixed them. |
Adding the ability to specify, on
@Timed
annotations, the service level objectives.Using seconds as the unit of time for these values.