sthetland commented on a change in pull request #9114:
URL: https://github.com/apache/druid/pull/9114#discussion_r438284123



##########
File path: docs/configuration/index.md
##########
@@ -353,6 +353,7 @@ The following monitors are available:
 |`org.apache.druid.segment.realtime.RealtimeMetricsMonitor`|Reports statistics 
on Realtime processes.|
 |`org.apache.druid.server.metrics.EventReceiverFirehoseMonitor`|Reports how 
many events have been queued in the EventReceiverFirehose.|
 |`org.apache.druid.server.metrics.QueryCountStatsMonitor`|Reports how many 
queries have been successful/failed/interrupted.|
+|`org.apache.druid.server.metrics.TaskCountStatsMonitor`|Reports how many 
tasks are success/failed/running/pending/waiting.|

Review comment:
       ... or "have been successful", to be consistency with the prior row in 
the table. 
   
   Even better though might be to expand out, for both rows: 
   
   ```
   |`org.apache.druid.server.metrics.QueryCountStatsMonitor`|Reports how many 
queries have succeeded, failed, or were interrupted.|
   |`org.apache.druid.server.metrics.TaskCountStatsMonitor`|Reports how many 
tasks have succeeded, failed, or are currently in the running, pending, or 
waiting state.|
   ``` 




----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to