You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by sl...@apache.org on 2014/03/27 16:23:42 UTC

[3/5] git commit: Fix BB relative get

Fix BB relative get


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

Branch: refs/heads/trunk
Commit: 180bb757164ae1c938ac0335ea4cfc83b54bd740
Parents: e615960
Author: Sylvain Lebresne <sy...@datastax.com>
Authored: Thu Mar 27 16:21:21 2014 +0100
Committer: Sylvain Lebresne <sy...@datastax.com>
Committed: Thu Mar 27 16:21:30 2014 +0100

----------------------------------------------------------------------
 src/java/org/apache/cassandra/db/marshal/BooleanType.java | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/180bb757/src/java/org/apache/cassandra/db/marshal/BooleanType.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/db/marshal/BooleanType.java b/src/java/org/apache/cassandra/db/marshal/BooleanType.java
index ca11cc2..d5f44ce 100644
--- a/src/java/org/apache/cassandra/db/marshal/BooleanType.java
+++ b/src/java/org/apache/cassandra/db/marshal/BooleanType.java
@@ -24,8 +24,13 @@ import org.apache.cassandra.serializers.TypeSerializer;
 import org.apache.cassandra.serializers.BooleanSerializer;
 import org.apache.cassandra.serializers.MarshalException;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 public class BooleanType extends AbstractType<Boolean>
 {
+    private static final Logger logger = LoggerFactory.getLogger(BooleanType.class);
+
     public static final BooleanType instance = new BooleanType();
 
     BooleanType() {} // singleton
@@ -38,8 +43,8 @@ public class BooleanType extends AbstractType<Boolean>
             return 1;
 
         // False is 0, True is anything else, makes False sort before True.
-        byte b1 = o1.get();
-        byte b2 = o2.get();
+        byte b1 = o1.get(o1.position());
+        byte b2 = o2.get(o2.position());
         if (b1 == 0)
             return b2 == 0 ? 0 : -1;
         return b2 == 0 ? 1 : 0;