[GitHub] [accumulo-website] EdColeman commented on a change in pull request #226: Doc update for ZooKeeper stat command config for Monitor Overview

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

EdColeman 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_r403031412
 
 

 ##########
 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:
   I think your points are valid - the issue that I had in creating this was were would I go, if on an upgrade, the monitor stopped showing the ZooKeeper info.  The configuration section seemed like a place I might go first - the monitor pages seems to provide more general info that existing users may skip - but, it is a monitor issue, so, maybe there?  And rather than duplicate info, the link.
   
   The essence I what I wanted to convey is that the monitor relies on the stat command.  Other ZooKeeper operations are using a connector and will continue to work - but the monitor is using an alternate connection path and that may not be obvious users.  With the zookeeper white list changes, they need to do update their existing  zookeeper configuration. 
   
   Fixing uno helps new instance experiences - what I hoped to convey was that exiting users may need adjustments - and those might not have been obvious 

----------------------------------------------------------------
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


With regards,
Apache Git Services