Skip to content

Comments

Added correct thread naming to compactor threads#6144

Merged
ddanielr merged 2 commits intoapache:2.1from
ddanielr:bugfix/fix-compactor-thread-naming
Feb 24, 2026
Merged

Added correct thread naming to compactor threads#6144
ddanielr merged 2 commits intoapache:2.1from
ddanielr:bugfix/fix-compactor-thread-naming

Conversation

@ddanielr
Copy link
Contributor

Sets the thread name prefix based on the compaction type instead of defaulting to MajC.

I looked at using the existing CompactionType enum but it's in the public API.

public abstract class ActiveCompaction {
public static enum CompactionType {
/**
* compaction to flush a tablets memory
*/
MINOR,
/**
* Accumulo no longer does merging minor compactions.
*/
@Deprecated(since = "2.1.0", forRemoval = true)
MERGE,
/**
* compaction that merges a subset of a tablets files into one file
*/
MAJOR,
/**
* compaction that merges all of a tablets files into one file
*/
FULL
}

The goal is to name the MinC and MajC threads correctly for troubleshooting.

Sets the thread name based on the compaction type instead of defaulting
to MajC.
@ddanielr ddanielr added this to the 2.1.5 milestone Feb 23, 2026
Compactors are single threaded and only run major compactions so MajC
thread names are still accurate.
@ddanielr ddanielr merged commit bd9c2c3 into apache:2.1 Feb 24, 2026
8 checks passed
@ddanielr ddanielr deleted the bugfix/fix-compactor-thread-naming branch February 24, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants