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

Sandeep Tata commented on CASSANDRA-502:
----------------------------------------

For the simple case of a 1 node cluster (replication = 1) where you're 
bootstrapping a second node, the existing code should breaks.
I get an NPE in the bootstrapper thread and the bootstrap actually fails. 
(Jonathan -- did this case work for you with trunk?) 

This patch makes sure you're picking up the nodes that handle new ranges 
correctly by using constructRangeToEndPointMap(newRanges, 
newTokenToEndPointMap); instead of the old tokenToEndPointMap which has the new 
nodes removed by the time you get here.

> Improve BootStrapperTest
> ------------------------
>
>                 Key: CASSANDRA-502
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-502
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.5
>         Environment: all
>            Reporter: Sandeep Tata
>            Assignee: Sandeep Tata
>            Priority: Minor
>             Fix For: 0.5
>
>         Attachments: CASSANDRA-502-v1.patch
>
>
> BootStrapper test should insert the tokens as "bootstrapping" so 
> tokenMetadata is in a "real" state before running getRangesWithSourceTarget().

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to