[CASSANDRA-994] re-organize endpointsnitch implementations

• Type: Task
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 0.7 beta 1
• Assignee: Erick Tryzelaar
• Reporter: Jonathan Ellis

Authors

Erick Tryzelaar

• Created: 16/Apr/10 17:44
• Updated: 16/Apr/19 09:33
• Resolved: 27/Apr/10 17:07

Description

I think the EPS implementations should look like this:

IEndPointSnitch: rename to IEndpointSnitch (lowercase P), otherwise fine as it is
SimpleSnitch: new default snitch. Implement IEPS directly, with sorting as no-ops.
AbstractEndpointSnitch: rename to AbstractRackAwareSnitch; replace isOnSameRack and isInSameDataCenter with getRack and getDatacenter (replacing getLocation).
EndPointSnitch: rename to RackInferringSnitch
PropertyFileEndpointSnitch: rename to PropertyFileSnitch, move from contrib to src
DatacenterEndpointSnitch: remove (redundant wrt PFS). "quorum factor per DC" needs to go in the ReplicationStrategy, not the snitch.

The patch sequence looks like:

01: perform renames of IEPS, AES, EPS, and PFES
02: add SS and make method changes to ARAS [was AES]
Separate ticket: DatacenterReplicationStrategy code changes