Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Mon, 27 Mar 2017 15:38:41 +0000 (UTC)
From: "Jeremiah Jordan (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.13059334.1490606804000.132682.1490629121686@Atlassian.JIRA>
In-Reply-To: <JIRA.13059334.1490606804000@Atlassian.JIRA>
References: <JIRA.13059334.1490606804000@Atlassian.JIRA> <JIRA.13059334.1490606804170@jira-lw-us.apache.org>
Subject: [jira] [Commented] (CASSANDRA-13382) cdc column addition strikes
 again
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Mon, 27 Mar 2017 15:38:46 -0000


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

Jeremiah Jordan commented on CASSANDRA-13382:
---------------------------------------------

+1 Looks good to me.  addViewToSchemaMutation calls addTableParamsToRowBuilder which was covered by the fix in CASSANDRA-12236.  So the only MV cdc thing that still needs catching is in makeUpdateForSchema which this patch covers.

> cdc column addition strikes again
> ---------------------------------
>
>                 Key: CASSANDRA-13382
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13382
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 3.11.x
>
>
> This is a followup of CASSANDRA-12697, where the patch mistakenly only handled the {{system_schema.tables}} table, while the {{cdc}} column has been added to {{system_schema.views}} table.
> The patch is pretty trivial, though this highlight that we don't seem to have upgrade tests for materialized views.


--
This message was sent by Atlassian JIRA
(v6.3.15#6346)