[GitHub] [kafka] hachikuji commented on a diff in pull request #12679: Adding KRaft Monitoring Related Metrics to docs/ops.html

[GitBox <gi...@apache.org>]

hachikuji commented on code in PR #12679:
URL: https://github.com/apache/kafka/pull/12679#discussion_r979099254


##########
docs/ops.html:
##########
@@ -1815,6 +1815,194 @@ <h4 class="anchor-heading"><a id="remote_jmx" class="anchor-link"></a><a href="#
       </tr>
   </tbody></table>
 
+<h4 class="anchor-heading"><a id="kraft_monitoring" class="anchor-link"></a><a href="#kraft_monitoring">KRaft Monitoring Metrics</a></h4>
+The set of metrics that allow monitoring of the KRaft quorum and the metadata log
+<h5 class="anchor-heading"><a id="kraft_controller_monitoring" class="anchor-link"></a><a href="#kraft_controller_monitoring">KRaft Controller Monitoring Metrics</a></h5>

Review Comment:
   Do you think it's worth pointing out somewhere that a controller means any node which has "controller" included among `process.roles`?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org