ddanielr opened a new issue, #3981:
URL: https://github.com/apache/accumulo/issues/3981

   **Is your feature request related to a problem? Please describe.**
   There are a number of compaction service properties that are required for 
the system to function. They have been placed in `Property.java` with default 
values that work for a given system.
   
   This design choice causes unnecessary complications when attempting to 
modify compaction service code; especially while supporting deprecation in one 
accumulo version and addition of a new property since both sets of properties 
cannot be defined at the same time.  
   
   This is further compounded by the fact that they are parsed in sections by 
the `DefaultCompactionPlanner`. This parsing means no direct references to 
these properties exist in the code base. This lack of references could cause 
someone to assume these properties could be removed without issue.
   
   **Describe the solution you'd like**
   Move these properties to the 
[accumulo.properties](https://github.com/apache/accumulo/blob/main/assemble/conf/accumulo.properties)
 file and assign them their current default values.
   
   This ensures that the properties are exposed to the user in a more direct 
method while providing us flexibility in our code base to change compaction 
service code.


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