From 02030dd658e6ca0cdb8987fd72e0454066e1b6d6 Mon Sep 17 00:00:00 2001 From: Sylvain Lebresne Date: Mon, 16 Jul 2012 15:34:33 +0200 Subject: [PATCH] Fix LCS bug with sstables containing only 1 row patch by slebresne; reviewed by jbellis for CASSANDRA-4411 --- CHANGES.txt | 1 + src/java/org/apache/cassandra/dht/Bounds.java | 5 ++++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGES.txt b/CHANGES.txt index f9b45d616c..a3ba123e0e 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -11,6 +11,7 @@ * (cql3) delete "component_index" column on DROP TABLE call (CASSANDRA-4420) * change nanoTime() to currentTimeInMillis() in schema related code (CASSANDRA-4432) * add a token generation tool (CASSANDRA-3709) + * Fix LCS bug with sstable containing only 1 row (CASSANDRA-4411) Merged from 1.0: * allow dropping columns shadowed by not-yet-expired supercolumn or row tombstones in PrecompactedRow (CASSANDRA-4396) diff --git a/src/java/org/apache/cassandra/dht/Bounds.java b/src/java/org/apache/cassandra/dht/Bounds.java index 0e4dbdf7f2..b5633c81a8 100644 --- a/src/java/org/apache/cassandra/dht/Bounds.java +++ b/src/java/org/apache/cassandra/dht/Bounds.java @@ -47,7 +47,10 @@ public class Bounds extends AbstractBounds public boolean contains(T position) { - return Range.contains(left, right, position) || left.equals(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)); } public Pair, AbstractBounds> split(T position)