You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sylvain Lebresne (JIRA)" <ji...@apache.org> on 2016/02/10 16:25:18 UTC

[jira] [Commented] (CASSANDRA-11050) SSTables not loaded after dropping column

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

Sylvain Lebresne commented on CASSANDRA-11050:
----------------------------------------------

I took a few minutes to upgrade the patch (below) for this with what's above and to write a quick dtest (PR [here|https://github.com/riptano/cassandra-dtest/pull/800]), I hope you don't mind [~amorton].

|| patch || utests || dtests ||
| [11050-3.0|https://github.com/pcmanus/cassandra/commits/11050-3.0] | [utests|http://cassci.datastax.com/view/Dev/view/pcmanus/job/pcmanus-11050-3.0-testall/lastBuild/] | [dtests|http://cassci.datastax.com/view/Dev/view/pcmanus/job/pcmanus-11050-3.0-dtest/lastBuild/] |
| [11050-trunk|https://github.com/pcmanus/cassandra/commits/11050-trunk] | [utests|http://cassci.datastax.com/view/Dev/view/pcmanus/job/pcmanus-11050-trunk-testall/lastBuild/] | [dtests|http://cassci.datastax.com/view/Dev/view/pcmanus/job/pcmanus-11050-trunk-dtest/lastBuild/] |

> SSTables not loaded after dropping column
> -----------------------------------------
>
>                 Key: CASSANDRA-11050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11050
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Distributed Metadata
>            Reporter: amorton
>            Assignee: amorton
>         Attachments: 11050-3.0.patch
>
>
> The {{system_schema.dropped_tables}} table is not flushed when schema is updated, this can result in SSTables not being loaded at startup and failure to start if the commit log contains mutations with the column. 
> Reproduce on cassandra-3.0 branch by starting a node and running following in cqlsh:
> {code}
> create keyspace dev WITH replication = {'class':'SimpleStrategy', 'replication_factor':1};
> use dev;
> create table foo (
>      foo text primary key,
>      bar text,
>      baz text
> );
> insert into foo (foo, bar, baz) values ('foo','this is bar', 'this is baz');
> alter table foo 
> drop baz;
> {code}
> Stop the node and restart, the following errors are raised and the node does not start:
> {code}
> ERROR 16:38:19 Exception in thread Thread[SSTableBatchOpen:1,5,main]
> java.lang.RuntimeException: Unknown column baz during deserialization
> 	at org.apache.cassandra.db.SerializationHeader$Component.toHeader(SerializationHeader.java:331) ~[main/:na]
> 	at org.apache.cassandra.io.sstable.format.SSTableReader.open(SSTableReader.java:485) ~[main/:na]
> 	at org.apache.cassandra.io.sstable.format.SSTableReader.open(SSTableReader.java:374) ~[main/:na]
> 	at org.apache.cassandra.io.sstable.format.SSTableReader$4.run(SSTableReader.java:533) ~[main/:na]
> 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[na:1.8.0_60]
> 	at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[na:1.8.0_60]
> 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) ~[na:1.8.0_60]
> 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_60]
> 	at java.lang.Thread.run(Thread.java:745) [na:1.8.0_60]
> ...
> ERROR 16:38:19 Exiting due to error while processing commit log during initialization.
> org.apache.cassandra.db.commitlog.CommitLogReplayer$CommitLogReplayException: Unexpected error deserializing mutation; saved to /var/folders/r2/rkv1jz3j0j74r9s1zm5xx9wc0000gn/T/mutation5408885979635225676dat.  This may be caused by replaying a mutation against a table with the same name but incompatible schema.  Exception follows: java.lang.RuntimeException: Unknown column baz during deserialization
> 	at org.apache.cassandra.db.commitlog.CommitLogReplayer.handleReplayError(CommitLogReplayer.java:633) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLogReplayer.replayMutation(CommitLogReplayer.java:556) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLogReplayer.replaySyncSection(CommitLogReplayer.java:509) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLogReplayer.recover(CommitLogReplayer.java:404) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLogReplayer.recover(CommitLogReplayer.java:151) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLog.recover(CommitLog.java:189) [main/:na]
> 	at org.apache.cassandra.db.commitlog.CommitLog.recover(CommitLog.java:169) [main/:na]
> 	at org.apache.cassandra.service.CassandraDaemon.setup(CassandraDaemon.java:283) [main/:na]
> 	at org.apache.cassandra.service.CassandraDaemon.activate(CassandraDaemon.java:551) [main/:na]
> 	at org.apache.cassandra.service.CassandraDaemon.main(CassandraDaemon.java:679) [main/:na]
> {code}
> {{dropped_columns}} is not in the list of tables to flush, {{SchemaKeyspace.ALL}}. 
> It's a simple patch to add it, attached. The fix will need to go to 3.0, 3.1 and trunk AFAIK
> however this will change the way the schema hash is calculated in {{SchemaKeyspace.calculateSchemaDigest()}} It looks like this would cause the nodes to announce a new version of the schema on (first) restart. 
> I  currently donit understand all the implications of changing the schema hash, thoughts ? 



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