apache / apache/accumulo

Move compaction service properties to accumulo props file

Open
#3,981 3 comments 0 reactions 1 assignee Claimed by @ddanielr View on GitHub
enhancement
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.