ctubbsii commented on a change in pull request #226: Doc update for ZooKeeper
stat command config for Monitor Overview
URL: https://github.com/apache/accumulo-website/pull/226#discussion_r402705301
##########
File path: _docs-2/administration/monitoring-metrics.md
##########
@@ -21,6 +21,11 @@ displaying various metrics over time. These include ingest
and scan performance
<img src="{{ site.baseurl }}/images/accumulo-monitor-1.png" alt="monitor
overview"/>
</a>
+Note: If the Zookeeper information is not available on the Overview, your
version of zookeeper may
+require setting the four-letter-word command whitelist to allow the stat
command.
+See [Configuration - Zookeeper Considerations] for more
+information.
Review comment:
* Should remove trailing whitespace.
* The ZooKeeper trademark has an uppercase Z and K (I know we're not
consistent everywhere on this, but I try to mention this whenever I see new doc
changes).
* Whitespace in Markdown link labels are handled inconsistently across
parsers, so it's best to avoid whitespace in them to reduce confusion and
potential breakage over time.
Also, I think this note can be summarized simply as:
```suggestion
Note: If the ZooKeeper information is not available on the Overview page,
you may need to
enable the `stat` [four-letter-word][zk-4lw] command, because the monitor
uses this command
to retrieve ZooKeeper information.
```
Then, you can link directly to the ZooKeeper's own instructions for how to
enable that command. The more succinctly we can keep this information, the less
likely it is to drift in accuracy over time.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
