You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jeff Jirsa (JIRA)" <ji...@apache.org> on 2017/03/27 16:27:41 UTC
[jira] [Comment Edited] (CASSANDRA-13382) cdc column addition
strikes again
[ https://issues.apache.org/jira/browse/CASSANDRA-13382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15943557#comment-15943557 ]
Jeff Jirsa edited comment on CASSANDRA-13382 at 3/27/17 4:27 PM:
-----------------------------------------------------------------
Minor nit that the comment is grammatically confusing (probably accidental if -> is):
{code}
+ * The tables to which we added the cdc column. This is used in {@link #makeUpdateForSchema} below to make sure we skip that
+ * column is cdc is disabled as the columns breaks pre-cdc to post-cdc upgrades (typically, 3.0 -> 3.X).
{code}
was (Author: jjirsa):
Minor nit that the comment is grammatically confusing (probably accidental if -> is):
{quote}
+ * The tables to which we added the cdc column. This is used in {@link #makeUpdateForSchema} below to make sure we skip that
+ * column is cdc is disabled as the columns breaks pre-cdc to post-cdc upgrades (typically, 3.0 -> 3.X).
{quote}
> 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)