[CASSANDRA-1191] Make Strategy classes fail if asked to fulfil an impossible replication request

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 0.7 beta 1
• Assignee: Matthew F. Dennis
• Reporter: Jonathan Ellis

Authors

Matthew F. Dennis

• Created: 14/Jun/10 22:04
• Updated: 16/Apr/19 09:33
• Resolved: 01/Jul/10 17:24

Description

I added to RSECT in CASSANDRA-1147,

// TODO DSS is asked to provide a total of 6 replicas, but we never give it 6 endpoints.
// thus we are testing undefined behavior, at best.

This is easiest to cause in DSS but it can afflict other Strategies too.

Let's update the ARS contract to state that calculateNaturalEndpoints (hence getNaturalEndpoints too) will throw IllegalStateException if more replicas are requested than the known endpoints (for DSS, this would apply on a per-DC basis) and amend the Strategy classes to check for this.

Issue Links

• relates to: CASSANDRA-1343 gossip throws IllegalStateException