sthetland commented on a change in pull request #11565: URL: https://github.com/apache/druid/pull/11565#discussion_r701504572
########## File path: docs/configuration/index.md ########## @@ -362,12 +362,11 @@ The following monitors are available: |----|-----------| |`org.apache.druid.client.cache.CacheMonitor`|Emits metrics (to logs) about the segment results cache for Historical and Broker processes. Reports typical cache statistics include hits, misses, rates, and size (bytes and number of entries), as well as timeouts and and errors.| |`org.apache.druid.java.util.metrics.SysMonitor`|Reports on various system activities and statuses using the [SIGAR library](https://github.com/hyperic/sigar). Requires execute privileges on files in `java.io.tmpdir`. Do not set `java.io.tmpdir` to `noexec` when using `SysMonitor`.| -|`org.apache.druid.server.metrics.HistoricalMetricsMonitor`|Reports statistics on Historical processes.| +|`org.apache.druid.server.metrics.HistoricalMetricsMonitor`|Reports statistics on Historical processes. Must be enabled on Historical process.| Review comment: Nit, but I'd reword this to say that this can be enabled only on historical processes, which conveys both that it must be enabled there and trying to do so on other processes will produce errors. ```suggestion |`org.apache.druid.server.metrics.HistoricalMetricsMonitor`|Reports statistics on Historical processes. Available only on Historical processes.| ``` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
