kevinrr888 opened a new issue, #5693: URL: https://github.com/apache/accumulo/issues/5693
**Is your feature request related to a problem? Please describe.** The FIXED_PROPERTIES set in the Property class is not maintainable. #5632 greatly increased the size of this set when exploring all properties to determine if they are fixed or not. This demonstrates that this set is not always considered when adding new properties (and/or existing properties weren't all considered when this set was created). **Describe the solution you'd like** Property should have a new field `isFixed` and the Property constructor should take this argument. This will force `isFixed` to be considered when adding new properties. **Describe alternatives you've considered** A potential alternative or temporary solution could be to add a test to PropertyTest which fails whenever new properties are added, prompting developers to consider if the property they added should be added to FIXED_PROPERTIES. Developers could then update the test after consideration. This change should target 4.0.0, however the alternative solution could target 2.1 So, the approach to fix this issue could be: * The new test in PropertyTest in 2.1 and main * The new `isFixed` field in main, can remove the test in main with this change -- 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]
