[ 
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)

Reply via email to