cql-rb 1.0.2 → 1.0.3

data/lib/cql/protocol/decoding.rb

@@ -38,8 +38,14 @@ module Cql
       end
 
       def read_long!(buffer)
-        top, bottom = buffer.read(8).unpack(Formats::TWO_INTS_FORMAT)
-        (top << 32) | bottom
+        hi, lo = buffer.read(8).unpack(Formats::TWO_INTS_FORMAT)
+        if (hi > 0x7fffffff)
+          hi ^= 0xffffffff
+          lo ^= 0xffffffff
+          0 - (hi << 32) - lo - 1
+        else
+          (hi << 32) + lo
+        end
       rescue RangeError => e
         raise DecodingError, e.message, e.backtrace
       end
@@ -57,7 +63,11 @@ module Cql
       end
 
       def read_int!(buffer)
-        buffer.read_int
+        val = buffer.read_int
+        if (val > 0x7fffffff)
+          val = 0 - ((val - 1) ^ 0xffffffff)
+        end
+        val
       rescue RangeError => e
         raise DecodingError, "Not enough bytes available to decode an int: #{e.message}", e.backtrace
       end

data/lib/cql/version.rb

@@ -1,5 +1,5 @@
 # encoding: utf-8
 
 module Cql
-  VERSION = '1.0.2'.freeze
+  VERSION = '1.0.3'.freeze
 end

data/spec/cql/protocol/decoding_spec.rb

@@ -79,11 +79,16 @@ module Cql
       end
 
       describe '#read_long!' do
-        it 'decodes a long' do
+        it 'decodes a positive long' do
           buffer = ByteBuffer.new("\x00\x00\xca\xfe\xba\xbe\x00\x00")
           Decoding.read_long!(buffer).should == 0x0000cafebabe0000
         end
 
+        it 'decodes a negative long' do
+          buffer = ByteBuffer.new("\xff\xff\xff\xff\xff\xff\xff\xff")
+          Decoding.read_long!(buffer).should == -1
+        end
+
         it 'consumes the bytes' do
           buffer = ByteBuffer.new("\xca\xfe\xba\xbe\xca\xfe\xba\xbe\xca\xfe\xba\xbe")
           Decoding.read_long!(buffer)
@@ -137,10 +142,15 @@ module Cql
           ByteBuffer.new("\x00\xff\x00\xff")
         end
 
-        it 'decodes an int' do
+        it 'decodes a positive int' do
           Decoding.read_int!(buffer).should == 0x00ff00ff
         end
 
+        it 'decodes a negative int' do
+          buffer = ByteBuffer.new("\xff\xff\xff\xff")
+          Decoding.read_int!(buffer).should == -1
+        end
+
         it 'consumes the bytes' do
           buffer << "\xab\xcd"
           Decoding.read_int!(buffer)

data/spec/integration/protocol_spec.rb

@@ -285,6 +285,76 @@ describe 'Protocol parsing and communication' do
           end
         end
 
+        it 'decodes positive counters' do
+          in_keyspace_with_counters_table do
+            max_long = (1 << 63) -1
+            response = query(%<UPDATE counters SET c1 = c1 + 1, c2 = c2 +#{max_long} WHERE id = 'positive'>)
+            response = query(%<SELECT * FROM counters>, :quorum)
+            response.rows.should == [
+              {'id' => 'positive', 'c1' => 1, 'c2' => max_long},
+            ]
+          end
+        end
+
+        it 'decodes negative counters' do
+          in_keyspace_with_counters_table do
+            min_long = 0 - (1 << 63)
+            response = query(%<UPDATE counters SET c1 = c1 - 1 , c2 = c2 -#{min_long.abs} WHERE id = 'negative'>)
+            response = query(%<SELECT * FROM counters>, :quorum)
+            response.rows.should == [
+              {'id' => 'negative', 'c1' => -1 , 'c2' => min_long}
+            ]
+          end
+        end
+
+        it 'decodes positive bigints' do
+          in_keyspace do
+            max_long = (1 << 63) -1
+            response = query('CREATE TABLE stuff (id VARCHAR, c1 bigint, c2 bigint, PRIMARY KEY (id))')
+            response = query(%<UPDATE stuff SET c1 = 1, c2 = #{max_long} WHERE id = 'stuff'>)
+            response = query(%<SELECT * FROM stuff>, :quorum)
+            response.rows.should == [
+              {'id' => 'stuff', 'c1' => 1, 'c2' => max_long}
+            ]
+          end
+        end
+
+        it 'decodes negative bigints' do
+          in_keyspace do
+            min_long = 0 - (1 << 63)
+            response = query('CREATE TABLE stuff (id VARCHAR, c1 bigint, c2 bigint, PRIMARY KEY (id))')
+            response = query(%<UPDATE stuff SET c1 = -1, c2 = #{min_long} WHERE id = 'stuff'>)
+            response = query(%<SELECT * FROM stuff>, :quorum)
+            response.rows.should == [
+              {'id' => 'stuff', 'c1' => -1, 'c2' => min_long}
+            ]
+          end
+        end
+
+        it 'decodes positive ints' do
+          in_keyspace do
+            max_int = (1 << 31) -1
+            response = query('CREATE TABLE stuff (id VARCHAR, c1 int, c2 int, PRIMARY KEY (id))')
+            response = query(%<UPDATE stuff SET c1 = 1, c2 = #{max_int} WHERE id = 'stuff'>)
+            response = query(%<SELECT * FROM stuff>, :quorum)
+            response.rows.should == [
+              {'id' => 'stuff', 'c1' => 1, 'c2' => max_int}
+            ]
+          end
+        end
+
+        it 'decodes negative ints' do
+          in_keyspace do
+            min_int = 0 - (1 << 31)
+            response = query('CREATE TABLE stuff (id VARCHAR, c1 int, c2 int, PRIMARY KEY (id))')
+            response = query(%<UPDATE stuff SET c1 = -1, c2 = #{min_int} WHERE id = 'stuff'>)
+            response = query(%<SELECT * FROM stuff>, :quorum)
+            response.rows.should == [
+              {'id' => 'stuff', 'c1' => -1, 'c2' => min_int}
+            ]
+          end
+        end
+
         it 'sends a DELETE command' do
           in_keyspace_with_table do
             response = query(%<DELETE email FROM users WHERE user_name = 'sue'>)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cql-rb
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-28 00:00:00.000000000 Z
+date: 2013-08-04 00:00:00.000000000 Z
 dependencies: []
 description: A pure Ruby CQL3 driver for Cassandra
 email:
@@ -112,7 +112,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1249323272134988131
+      hash: -3215276701075097941
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23