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

Add Support for @MeterTag to @Counted #4725

Closed
ColdFireIce opened this issue Feb 5, 2024 · 2 comments · Fixed by #4958
Closed

Add Support for @MeterTag to @Counted #4725

ColdFireIce opened this issue Feb 5, 2024 · 2 comments · Fixed by #4958
Labels
enhancement A general enhancement help wanted An issue that a contributor can help us with module: micrometer-core An issue that is related to our core module
Milestone

Comments

@ColdFireIce
Copy link

Hi there,

I would like to count method calls with an enum in the parameters and use these as tags for the counter. Something analogue to to the @timed annotation.
It would be super to have the option to add an MeterAnnotationHandler to the CountedAspect, as you can for the TimedAspect.

I'm not sure if this needs more explanation or an example?

Best regards

@shakuzen shakuzen added enhancement A general enhancement module: micrometer-core An issue that is related to our core module and removed waiting-for-triage labels Feb 6, 2024
@shakuzen shakuzen added this to the 1.x milestone Feb 6, 2024
@shakuzen shakuzen added the help wanted An issue that a contributor can help us with label Feb 6, 2024
@shakuzen
Copy link
Member

shakuzen commented Feb 6, 2024

Thanks for the enhancement request. I think it makes sense to bring CountedAspect support up to the level of TimedAspect with respect to @MeterTag. Is this something you'd be willing to submit a pull request for?

izeye added a commit to izeye/micrometer that referenced this issue Apr 10, 2024
@izeye
Copy link
Contributor

izeye commented Apr 10, 2024

I created #4958 to try to resolve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A general enhancement help wanted An issue that a contributor can help us with module: micrometer-core An issue that is related to our core module
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants