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 JMX note for Kafka consumer metrics #19446
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 PR. I am wondering about rephrasing the note. WDYT?
@@ -1644,7 +1644,7 @@ Spring Boot registers the following core metrics when applicable: | |||
** Number of classes loaded/unloaded | |||
* CPU metrics | |||
* File descriptor metrics | |||
* Kafka consumer metrics | |||
* Kafka consumer metrics (`spring.jmx.enabled` must be set to `true`) |
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.
This is a good idea. I'd rather change the note to something like "JMX support should be enabled" or something like that with a link to the section that explains how to do it rather than the reference to the property.
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.
@snicoll Thanks for the feedback! I changed it as suggested.
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 quick feedback. I meant that the text on "JMX support" should be a link to the relevant section. Have you missed it or do you disagree with that approach?
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.
@snicoll Sorry. I overlooked your comment. I changed it again. Please let me know if it isn't what you meant.
6208617
to
f109bdd
Compare
f109bdd
to
5c5b4f3
Compare
Looking at #19348, it seems to be good to add a JMX note for Kafka consumer metrics similar to Tomcat metrics.