dlmarion commented on PR #6462:
URL: https://github.com/apache/accumulo/pull/6462#issuecomment-4892726917

   > I do not have a great idea how much this will affect things in a 
real-world deployment so if anyone has an insights/opinions on things here that 
would be helpful.
   
   ZooKeeper has a maximum size setting for the data on a node. By default, 
it's 1MB. If you attempt to write to a node and are over the limit, then the 
write will fail. The user will need to increase the limit on all ZK servers and 
clients, so restarting all of their ZK and Accumulo processes. For this reason 
I don't think we should do anything in the upgrade code.
   
   I would be curious if you can cause this failure in an IT by setting every 
possible property to a long, but still reasonable value. 
   
   
   > It is probably simpler, and satisfies user interest better, to just log 
configuration changes on the server-side, when an RPC request comes in to 
change the config.
   
   I agree with this. Maybe it's possible to add this to the existing audit 
logging?


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to