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_r402707423
##########
File path: _docs-2/administration/monitoring-metrics.md
##########
@@ -112,3 +117,4 @@ Below are additional resources for configuring Metrics2:
[grafana-post]: {{ site.baseurl }}/blog/2018/03/22/view-metrics-in-grafana.html
[metrics2-javadoc]:
https://hadoop.apache.org/docs/current/api/org/apache/hadoop/metrics2/package-summary.html
[tracing]: {% durl troubleshooting/tracing %}
+[Configuration - Zookeeper Considerations]: {% durl
configuration/overview#monitor-and-zookeeper-four-letter-word-command-whitelist
%}
Review comment:
The link directly to ZK's own docs on the matter is far more useful than
jumping around our own docs, and helps us avoid taking on the responsibility of
being a configuration guide for ZooKeeper itself.
```suggestion
[zk-4lw]: https://zookeeper.apache.org/doc/r3.5.7/zookeeperAdmin.html#sc_4lw
```
----------------------------------------------------------------
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
