ddanielr commented on issue #3981: URL: https://github.com/apache/accumulo/issues/3981#issuecomment-1890130267
I tried just emptying the default compaction values for these properties and ran into a LOT of validation errors that were only solved by poking holes in the current validation and test code. We would run into those problems even if these properties fell under the general/custom prefixes. I'm happy to create a separate PR with the changes needed for empty property definitions in `Property.java` to compare the differences. This may lead to refactoring Properties to support some sort of `example` flag/option that doesn't result in reducing our ability to validate properties. I agree that having the empty values in Property.java is a better goal as it allows us to use the PropertyType.json and have stronger property validation. Also automatic property documentation is generated from the entries in `Property.java`. -- 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]
