cassandra-commits mailing list archives

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


Branimir Lambov commented on CASSANDRA-12990:

Since we don't get the rack info, the new adapter should not [offer a means of getting it|]
at all. That method shouldn't be called by the no-replication allocator, let's make that clear
there and then by adding a comment and throwing.

The adapter should also just return 1 for the replica count, so that the [0 check in {{TokenAllocatorFactory}}|]
can be removed and the 0 replica hack is only done in one place in the code.

Now that {{waitForGossipToSettle}} is in {{Gossiper}}, the name is somewhat redundant. Rename
to {{waitToSettle}}?

> 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