[GitHub] [druid] sthetland commented on a change in pull request #9114: add docs for TaskCountStatsMonitor

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

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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org