Move compaction service properties to accumulo props file
- Dominant language
- Java
- Stars
- 1.2k
- Forks
- 487
- Avg merge
- 4d 5h
- Merged PRs (30d)
- 13
Description
**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.
Contributor guide
Assessment
This issue has not been assessed yet.