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