[CASSANDRA-14621] Refactor CompactionStrategyManager

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 4.0-alpha1, 4.0
• Component/s: Local/Compaction
• Assignee: Blake Eggleston
• Reporter: Blake Eggleston

Authors

Blake Eggleston

Reviewers

Marcus Eriksson

• Created: 03/Aug/18 21:30
• Updated: 15/May/20 08:00
• Resolved: 24/Aug/18 15:24

Description

CompactionStrategyManager grew a decent amount of duplicated code as part of CASSANDRA-9143, which added pendingRepairs alongside the repaired and unrepaired buckets. At this point, the logic that routes sstables between the different buckets, and the different partition range divisions has gotten a little complex, and editing it is tedious and error prone. With transient replication requiring yet another bucket for, this seems like a good time to split some of the functionality of CSM into other classes, and make sstable routing a bit more generalized.