keith-turner opened a new issue, #6216:
URL: https://github.com/apache/accumulo/issues/6216

   **Is your feature request related to a problem? Please describe.**
   
   Started some Accumulo servers processes with that changes in #6207 that 
removed port search properties.  The server processes still had port search 
properties in the config.  The properties were ignored and nothing complained 
about them.  Site config called this 
[code](https://github.com/apache/accumulo/blob/c01b34949d9c2d9a4eddd33ce8bd62b22a0fbd79/core/src/main/java/org/apache/accumulo/core/conf/ConfigCheckUtil.java#L55)
 and I suspect it ignored the port search properties because they had something 
like a `tserver` prefix.
   
   **Describe the solution you'd like**
   
   Warn on properties that will never be used.  Some prefixes in the accumulo 
config are meant to be extended by user and others are not.  Maybe an 
annotation could be added to prefixes to denote if they can be extended or not. 
 For example the `tserver.` prefix could have an annotation that specifies it 
can not be extended and the `table.iterator` prefix could have an annotation 
that specifies it can be extended.  Probably only need one of the annotations, 
can treat absence as the opposite.
   
   With this change the code could know that only having the `tserver.` prefix 
does not make a property valid.
   


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