Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Thu, 10 Nov 2016 14:29:58 +0000 (UTC)
From: "T Jake Luciani (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.13019557.1478702940000.244576.1478788198476@Atlassian.JIRA>
In-Reply-To: <JIRA.13019557.1478702940000@Atlassian.JIRA>
References: <JIRA.13019557.1478702940000@Atlassian.JIRA> <JIRA.13019557.1478702940272@arcas>
Subject: [jira] [Commented] (CASSANDRA-12888) Incremental repairs broken for
 MVs and CDC
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 10 Nov 2016 14:30:00 -0000


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

T Jake Luciani commented on CASSANDRA-12888:
--------------------------------------------

bq. Once CFs have been retrieved for a KS, no filtering for MVs seem to happen and they'll be repaired just as regular tables. I'm not sure if that's intentional. 

This was intentional, though you really shouldn't repair the whole KS with MVs in it, but rather just the base tables.  You can repair only the views in the case of a corrupt/lost sstable in the view since it would be faster (as it wouldn't go through the memtable).

> Incremental repairs broken for MVs and CDC
> ------------------------------------------
>
>                 Key: CASSANDRA-12888
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12888
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Streaming and Messaging
>            Reporter: Stefan Podkowinski
>            Priority: Critical
>
> SSTables streamed during the repair process will first be written locally and afterwards either simply added to the pool of existing sstables or, in case of existing MVs or active CDC, replayed on mutation basis:
> As described in {{StreamReceiveTask.OnCompletionRunnable}}:
> {quote}
> We have a special path for views and for CDC.
> For views, since the view requires cleaning up any pre-existing state, we must put all partitions through the same write path as normal mutations. This also ensures any 2is are also updated.
> For CDC-enabled tables, we want to ensure that the mutations are run through the CommitLog so they can be archived by the CDC process on discard.
> {quote}
> Using the regular write path turns out to be an issue for incremental repairs, as we loose the {{repaired_at}} state in the process. Eventually the streamed rows will end up in the unrepaired set, in contrast to the rows on the sender site moved to the repaired set. The next repair run will stream the same data back again, causing rows to bounce on and on between nodes on each repair.
> See linked dtest on steps to reproduce. An example for reproducing this manually using ccm can be found [here|https://gist.github.com/spodkowinski/2d8e0408516609c7ae701f2bf1e515e8]


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