dlmarion commented on PR #5708: URL: https://github.com/apache/accumulo/pull/5708#issuecomment-3046637495
> My current thought is to remove `Property.FIXED_PROPERTIES` because almost everything is updateable and the code that enforced a static view was just removed. Instead, I'm thinking of adding an annotation to each Property to document the scope of the property. For example, rpc , instance, port properties, block cache properties, etc. have a PROCESS scope. Other properties, like MANAGER_BULK_TIMEOUT, may have a SESSION scope where they are checked each time. Maybe there are other values for other cases. If no annotation is supplied, then maybe there is a default value which means that its configurable. This approach continues to allow us to provide documentation, but if the annotation is wrong it's a documentation problem and doesn't affect the behavior of the code. 5a0c845 is an example of what I was trying to describe. I updated a bunch, but not all, of the properties with the PropertyScope annotation. -- 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
