cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dikang Gu (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-12990) More fixes to the TokenAllocator
Date Thu, 22 Dec 2016 23:47:58 GMT


Dikang Gu commented on CASSANDRA-12990:

[~blambov] thanks for the review! 

the [getRack |]
was not what I meant to, I think it comes from some rebase error. I was meant to have each
node in separate group, as I put in the comment.

I've addressed other comments, please take a look again, thanks!


> More fixes to the TokenAllocator
> --------------------------------
>                 Key: CASSANDRA-12990
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Dikang Gu
>            Assignee: Dikang Gu
>             Fix For: 3.12
> CASSANDRA-12983 fixes the NoReplicationTokenAllocator in the replication factor equals
0 case.
> We find one more problem that, in our big cluster, ~1000 nodes, the Gossip takes longer
time to settle down, so the TokenAllocator may throw NPE when trying to find the dc rack information
from the Gossip metadata, in the getStrategy function.
> This patch will fix it, it does two things:
> 1. wait gossip to settle down before trying to allocating tokens.
> 2. for replication factor equals 0 or 1 case, we do not need to check the topology for
rack information, since each node will be treated separately. 

This message was sent by Atlassian JIRA

View raw message