cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcus Eriksson (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CASSANDRA-8739) Don't check for overlap with sstables that have had their start positions moved in LCS
Date Thu, 05 Feb 2015 12:53:34 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-8739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307164#comment-14307164
] 

Marcus Eriksson edited comment on CASSANDRA-8739 at 2/5/15 12:52 PM:
---------------------------------------------------------------------

we pick a bunch of files for compaction (those are not currently compacting), then we make
sure that by compacting those into L1, we don't cause any overlap in L1. We do that by comparing
first/last keys in the candidates with the first/last keys of the sstables that are currently
compacting.


was (Author: krummas):
we pick a bunch of files for compaction (those are not currently compacting), then we make
sure that by compacting those into L1, we don't cause any overlap in L1. We do that by checking
first/last keys in the sstables

> Don't check for overlap with sstables that have had their start positions moved in LCS
> --------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-8739
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8739
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Marcus Eriksson
>            Assignee: Marcus Eriksson
>             Fix For: 2.1.3
>
>         Attachments: 0001-8739.patch
>
>
> When picking compaction candidates in LCS, we check that we won't cause any overlap in
the higher level. Problem is that we compare the files that have had their start positions
moved meaning we can cause overlap. We need to also include the tmplink files when checking
this.
> Note that in 2.1 overlap is not as big problem as earlier, if adding an sstable would
cause overlap, we send it back to L0 instead, meaning we do a bit more compaction but we never
actually have overlap.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message