From 35335caa034d5a22dd27058d011b1454fcc87850 Mon Sep 17 00:00:00 2001 From: Sylvain Lebresne Date: Mon, 16 Jul 2012 17:31:55 +0200 Subject: [PATCH] Follow up to #4411 patch by slebresne; reviewed by jbellis for CASSANDRA-4411 --- src/java/org/apache/cassandra/dht/Bounds.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/java/org/apache/cassandra/dht/Bounds.java b/src/java/org/apache/cassandra/dht/Bounds.java index b5633c81a8..c6378a7c51 100644 --- a/src/java/org/apache/cassandra/dht/Bounds.java +++ b/src/java/org/apache/cassandra/dht/Bounds.java @@ -47,10 +47,10 @@ public class Bounds extends AbstractBounds public boolean contains(T position) { - // Range.contains doesnt work correctly if left == right because for - // Range that means a wrapping range that select the whole ring. So we - // must explicitely handle this case - return left.equals(position) || (!left.equals(right) && Range.contains(left, right, position)); + // Range.contains doesnt work correctly if left == right (unless both + // are minimum) because for Range that means a wrapping range that select + // the whole ring. So we must explicitely handle this case + return left.equals(position) || ((right.isMinimum(partitioner) || !left.equals(right)) && Range.contains(left, right, position)); } public Pair, AbstractBounds> split(T position)