Updated Branches:
  refs/heads/trunk 4021fffa0 -> 6cc509463

Add sanity checks


Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/9ae960a1
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/9ae960a1
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/9ae960a1

Branch: refs/heads/trunk
Commit: 9ae960a1a4f57e3c9ec018f3cbb32fd3312d7a6e
Parents: 25a46ea
Author: Sylvain Lebresne <sylv...@datastax.com>
Authored: Wed Jul 24 14:18:57 2013 +0200
Committer: Sylvain Lebresne <sylv...@datastax.com>
Committed: Wed Jul 24 14:18:57 2013 +0200

----------------------------------------------------------------------
 .../db/index/AbstractSimplePerColumnSecondaryIndex.java         | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/9ae960a1/src/java/org/apache/cassandra/db/index/AbstractSimplePerColumnSecondaryIndex.java
----------------------------------------------------------------------
diff --git 
a/src/java/org/apache/cassandra/db/index/AbstractSimplePerColumnSecondaryIndex.java
 
b/src/java/org/apache/cassandra/db/index/AbstractSimplePerColumnSecondaryIndex.java
index 63af51b..2ff2d27 100644
--- 
a/src/java/org/apache/cassandra/db/index/AbstractSimplePerColumnSecondaryIndex.java
+++ 
b/src/java/org/apache/cassandra/db/index/AbstractSimplePerColumnSecondaryIndex.java
@@ -94,7 +94,9 @@ public abstract class AbstractSimplePerColumnSecondaryIndex 
extends PerColumnSec
         DecoratedKey valueKey = getIndexKeyFor(column.value());
         int localDeletionTime = (int) (System.currentTimeMillis() / 1000);
         ColumnFamily cfi = ColumnFamily.create(indexCfs.metadata);
-        cfi.addTombstone(makeIndexColumnName(rowKey, column), 
localDeletionTime, column.timestamp());
+        ByteBuffer name = makeIndexColumnName(rowKey, column);
+        assert name.remaining() > 0 && name.remaining() <= 
IColumn.MAX_NAME_LENGTH : name.remaining();
+        cfi.addTombstone(name, localDeletionTime, column.timestamp());
         indexCfs.apply(valueKey, cfi, SecondaryIndexManager.nullUpdater);
         if (logger.isDebugEnabled())
             logger.debug("removed index entry for cleaned-up value {}:{}", 
valueKey, cfi);
@@ -105,6 +107,7 @@ public abstract class AbstractSimplePerColumnSecondaryIndex 
extends PerColumnSec
         DecoratedKey valueKey = getIndexKeyFor(column.value());
         ColumnFamily cfi = ColumnFamily.create(indexCfs.metadata);
         ByteBuffer name = makeIndexColumnName(rowKey, column);
+        assert name.remaining() > 0 && name.remaining() <= 
IColumn.MAX_NAME_LENGTH : name.remaining();
         if (column instanceof ExpiringColumn)
         {
             ExpiringColumn ec = (ExpiringColumn)column;

Reply via email to