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:
[email protected]
With regards,
Apache Git Services
