[ https://issues.apache.org/jira/browse/CASSANDRA-3540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13163577#comment-13163577 ]
Sylvain Lebresne commented on CASSANDRA-3540: --------------------------------------------- The patch lgtm. Regarding the tests, I think it would be worth testing if we're correctly able to load old sstable, those without a saved partitioner, which I don't think the attached test does, does it (but the current test is good, we should keep it) ? Typically we could have a similar test than in the patch, but that nukes the statistics component of the sstables before reloading them . > Wrong check of partitioner for secondary indexes > ------------------------------------------------ > > Key: CASSANDRA-3540 > URL: https://issues.apache.org/jira/browse/CASSANDRA-3540 > Project: Cassandra > Issue Type: Bug > Components: Core > Affects Versions: 1.0.4 > Reporter: Sylvain Lebresne > Assignee: Yuki Morishita > Priority: Critical > Fix For: 1.0.6 > > Attachments: cassandra-1.0-3540-v2.txt, cassandra-1.0-3540.txt > > > CASSANDRA-3407 doesn't handle the fact that secondary indexes have a specific > partitioner (LocalPartitioner). This result in the following error when > starting nodes in 1.0.4: > {noformat} > java.lang.RuntimeException: Cannot open > /var/lib/cassandra/data/Index/AttractionLocationCategoryDateIdx.AttractionLocationCategoryDateIdx_09partition_idx-h-1 > because partitioner does not match org.apache.cassandra.dht.LocalPartitioner > {noformat} -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira