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_r403064982
##########
File path: _docs-2/configuration/overview.md
##########
@@ -86,13 +86,26 @@ client.tableOperations().setProperty("mytable",
"table.durability", "log");
### Zookeeper Considerations
+#### Storing Accumulo Server Properties
+
Review comment:
The main intent I have in my feedback is to keep the documentation concise.
We already have so much documentation, it's overwhelming and critical
information gets buried in pages and pages of superfluous explanatory text.
Every addition to the docs that focuses on one doc deficiency at a time risks
making the situation slightly worse, which builds up over time. I just want to
avoid that and try to think about the impact on the totality of the
documentation by keeping the additions concise.
> The essence I what I wanted to convey is that the monitor relies on the
stat command.
Right. I think that's the best thing to focus on. I just want it to be
concise.
As for where this should live, I think your original "Note" is fine (did my
proposed changes to that satisfy your intent in a concise way?). It could also
live in the "Troubleshooting" section.
----------------------------------------------------------------
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
