[GitHub] [accumulo] EdColeman commented on pull request #3015: Print warning in log when user does not have permission to get config

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

EdColeman commented on PR #3015:
URL: https://github.com/apache/accumulo/pull/3015#issuecomment-1274905406

   I am late to this, but it seems less important to display a value because that's what the users can see vs. what is set.  With the example above, if `table.bloom.enabled=true` was set at a level that the user cannot "see" - would it show that it was a) set to true somewhere, b) lie to the uses and say the value was "false", c) show that that value cannot be displayed because of permissions or d) just fail the entire command.
   
   It seems that c) or d) are the only valid choices - we cannot display a value that is not what is used - even if there's a warning at the beginning - so b should be right out.  Allowing a seems a way to get around permissions that might not be ideal, but at least reflects the actual state of the system.


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org