mansikhara opened a new pull request, #258:
URL: https://github.com/apache/cassandra-analytics/pull/258

   …nd replicas from Cassandra
   
   The bulk reader derived token ranges and replicas from node tokens and the 
replication factor, assuming racks are not in use. NetworkTopologyStrategy is 
rack aware, so with uneven racks the derived replicas can differ from 
Cassandra's and a read can return incomplete or stale data. CASSANALYTICS-197 
took the ranges from Sidecar's token-range-replicas endpoint only for keyspaces 
with transient replicas, as the ring held one token per node.
   
   Take them from Cassandra for every keyspace. A node owning several tokens is 
represented once, carrying its lowest token, as the reported ranges say which 
ranges it replicates. deriveTokenRanges falls back to the local derivation, and 
is ignored for keyspaces with transient replicas; it replaces 
forceCassandraTokenRanges, which is now the default. An incomplete 
token-range-replicas response fails with a clear message.
   
   Adds a data-level integration test on a two-datacenter cluster with uneven 
racks, where a replica that only rack-aware placement includes holds the newest 
value, and a vnode variant of the multi-rack placement checks. Builds on the 
approach in https://github.com/apache/cassandra-analytics/pull/149.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to