divijvaidya commented on code in PR #14331:
URL: https://github.com/apache/kafka/pull/14331#discussion_r1315692639


##########
docs/ops.html:
##########
@@ -1841,6 +1841,63 @@ <h4 class="anchor-heading"><a id="remote_jmx" 
class="anchor-link"></a><a href="#
       </tr>
   </tbody></table>
 
+<h4 class="anchor-heading"><a id="tiered_storage_monitoring" 
class="anchor-link"></a><a href="#tiered_storage_monitoring">Tiered Storage 
Monitoring Metrics</a></h4>
+  The following set of metrics are available for monitoring of the tiered 
storage feature:<br/><br/>
+  <table class="data-table">
+    <tbody>
+      <tr>
+        <th>Metric/Attribute name</th>
+        <th>Description</th>
+        <th>Mbean name</th>

Review Comment:
   This looks incorrect since this says that MBean name is in third column but 
the it's actually in second column. 
   
   It would help if you could please add a screenshot of how the website looks 
like with these changes with your next revision. You can find how to spin up 
website locally here: 
https://cwiki.apache.org/confluence/display/KAFKA/Setup+Kafka+Website+on+Local+Apache+Server
 
   
   Copy the contents of this file to kafka-site locally to see the changes.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to