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)