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
Metric API #1181
Metric API #1181
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.
I need more context here for some of the components.
</parent> | ||
<artifactId>feign-metrics5</artifactId> | ||
<name>Feign Metrics5</name> | ||
<description>Feign Dropwizard Metrics 5</description> |
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.
I'd like to suggest an alternative, Micrometer. It can wrap Dropwizard and will allow this to be integrated directly into Spring Boot/Cloud, since Micrometer is their preferred metrics collector.
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.
My plan is to add that one later, added metrics 5 first cause is the one I'm using and gets values for me faster
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.
They both do the same thing. If you want to have multiple implementations I’d like to suggest that we rename this to metrics-dropwizard
Provide a first-class Metrics API that users can tap into to gain insight into the request/response lifecycle.
My plan is:
Extend the capabilities to Async and Reactive (before 10.8 release)
Add metrics support for OpenTracing
After this, I feel we are in a good place to cut 10.8 and being 11 branch