[CASSANDRA-6688] Avoid possible sstable overlaps with leveled compaction

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 2.0.6
• Labels: lcs
• Severity: Normal
• Since Version: 2.0 beta 1
• Assignee: Marcus Eriksson
• Reporter: Marcus Eriksson

Authors

Marcus Eriksson

Reviewers

Jonathan Ellis

• Created: 11/Feb/14 06:31
• Updated: 16/Apr/19 09:31
• Resolved: 13/Feb/14 07:48

Description

Two cases where we can end up with overlapping sstables in the leveled manifest;

FIrst one is when we skip levels during compaction. Here we need to make sure we are not compacting in newLevel - 1 since if, for example, we are doing a L1 -> L2 compaction and then start a new L0 compaction where we decide to skip L1, we could have overlapping sstables in L2 when the compactions are done. This case is new in 2.0 since we check if we skip levels before the compaction starts.

Second case is where we try to include as many overlapping L0 sstables as possible, here we could add sstables that are not compacting, but overlap sstables that are.

Issue Links

• is duplicated by: CASSANDRA-6673 Compaction assertion error on beforeAppend test due to decommision - add node and/or overlapping sstables