dlmarion commented on issue #5695: URL: https://github.com/apache/accumulo/issues/5695#issuecomment-3028203342
I think the use for `isValidZooProperty` is to deny setting any property in ZooKeeper that is process specific. This would include `instance` properties and properties that the user might set in their site file for a specific instance of a process. For example, a user may want to use different block cache sizes on different servers due to a heterogeneous environment. Setting the block cache size in ZK, I think, would override these settings. Based on the current contents of the `isValidZooProperty` method, it looks like the intent is to allow everything but the instance properties. It looks like we allow setting other process properties (like block cache) in ZooKeeper, and leave it up to the user to make the correct decision for their deployment. -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
