[
https://issues.apache.org/jira/browse/CASSANDRA-11139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sylvain Lebresne updated CASSANDRA-11139:
-----------------------------------------
Assignee: Branimir Lambov
> New token allocator is broken
> -----------------------------
>
> Key: CASSANDRA-11139
> URL: https://issues.apache.org/jira/browse/CASSANDRA-11139
> Project: Cassandra
> Issue Type: Bug
> Reporter: Tommy Stendahl
> Assignee: Branimir Lambov
>
> It looks like the logic in the if statment in
> {{TokenAllocation.getStrategy()}} is broken.
> {code}
> if (replicas >= racks)
> {...}
> else if (racks == 1)
> {...}
> else
> throw new ConfigurationException(String.format("Token allocation failed:
> the number of racks %d in datacenter %s is lower than its replication factor
> %d.", racks, dc, replicas));
> {code}
> I think the first if statment should be {{(replicas <= racks)}}, otherwise
> this will be true if we have fewer racks then replicas and I guess that
> should result in an exception.
> Also the {{if}} and the {{else if}} blocks should change place, it should be
> {code}
> if (racks == 1)
> {...}
> else if (replicas <= racks)
> {...}
> else
> {code}
> If {{replicas==1}} and {{racks==1}} I think you want the {{StrategyAdapter}}
> from {{(racks==1)}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
