[ 
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

        

Reply via email to