rocksdb-ruby 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8e037812447e317397e2a8900ab252a869f90e00
-  data.tar.gz: fb7862fc46627bd157c2328269af4f887d4bcc3c
+  metadata.gz: a3516747ab80e3ed39c598b6c282790b75522da4
+  data.tar.gz: 721735897c276efa3a67ad636139bec399987b2f
 SHA512:
-  metadata.gz: f22299b9af97284ef06a413eda56b8fee76ec264b9f747b4c3a6377a31659cfcdd35e519e784ac99018b21d2dcb5aff165e361080d0ef5ec4c18ede30de7780c
-  data.tar.gz: 085cae8e95e91c87acbaa455cf3bf995d945b5d9d144c676d71c359a9080a393c1d63796366800d28431d7c8e830d0a6b6bacc5cf2efcb370253655b527ea603
+  metadata.gz: e0a62fffed025eaa9fe180029420bfb6ef5c77e74d735638c47875c147ee4c770e7ca73469a5d294b0ba2d1ac86f7f30b30c668383f57205b84b37955b187793
+  data.tar.gz: 59924a67d6ae19f360fa038a57f3f69124b0f0f2bf84b2f710d7be3425154ed74c5d04d700ed6d875f35e504e041f7dd9ea98acdabd64cdb68d0171df5cce919

data/ext/rocksdb/rocksdb_batch_rb.cc

@@ -17,8 +17,8 @@ extern "C" {
     Check_Type(v_value, T_STRING);
 
     rocksdb::WriteBatch *batch;
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
-    std::string value = std::string((char*)RSTRING_PTR(v_value));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
+    std::string value = std::string((char*)RSTRING_PTR(v_value), RSTRING_LEN(v_value));
     
     Data_Get_Struct(self, rocksdb::WriteBatch, batch);
     batch->Put(key, value);
@@ -28,7 +28,7 @@ extern "C" {
     Check_Type(v_key, T_STRING);
 
     rocksdb::WriteBatch *batch;
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
     
     Data_Get_Struct(self, rocksdb::WriteBatch, batch);
     batch->Delete(key);

data/ext/rocksdb/rocksdb_db_rb.cc

@@ -78,11 +78,10 @@ extern "C" {
     rocksdb_pointer* db_pointer;
     Data_Get_Struct(self, rocksdb_pointer, db_pointer);
     
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
-    std::string value = std::string((char*)RSTRING_PTR(v_value));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
+    std::string value = std::string((char*)RSTRING_PTR(v_value), RSTRING_LEN(v_value));
 
-    std::cout <<  "key " + key + "\n" ;
-    std::cout <<  "value " + value + "\n" ;
+    // std::cout << key << "\n";
     rocksdb::Status status = db_pointer->db->Put(rocksdb::WriteOptions(), key, value);
     
     return status.ok() ? Qtrue : Qfalse;
@@ -105,7 +104,7 @@ extern "C" {
     rocksdb_pointer* db_pointer;
     Data_Get_Struct(self, rocksdb_pointer, db_pointer);
     
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
     std::string value;
     rocksdb::Status status = db_pointer->db->Get(rocksdb::ReadOptions(), key, &value);    
 
@@ -128,7 +127,7 @@ extern "C" {
 
     for(i=0; i < length; i++){
       VALUE op = rb_ary_entry(v_array, i);
-      keys[i] = rocksdb::Slice((char*)RSTRING_PTR(op));
+      keys[i] = rocksdb::Slice((char*)RSTRING_PTR(op), RSTRING_LEN(op));
     }
 
     status = db_pointer->db->MultiGet(rocksdb::ReadOptions(),keys,&values);
@@ -144,7 +143,7 @@ extern "C" {
     rocksdb_pointer* db_pointer;
     Data_Get_Struct(self, rocksdb_pointer, db_pointer);
 
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
     rocksdb::Status status = db_pointer->db->Delete(rocksdb::WriteOptions(), key);
     
     return status.ok() ? Qtrue : Qfalse;
@@ -156,7 +155,7 @@ extern "C" {
     rocksdb_pointer* db_pointer;
     Data_Get_Struct(self, rocksdb_pointer, db_pointer);
 
-    std::string key = std::string((char*)RSTRING_PTR(v_key));
+    std::string key = std::string((char*)RSTRING_PTR(v_key), RSTRING_LEN(v_key));
     std::string value = std::string();
     
     return db_pointer->db->KeyMayExist(rocksdb::ReadOptions(), key, &value) ? Qtrue : Qfalse;

data/ext/rocksdb/rocksdb_iterator_rb.cc

@@ -24,7 +24,7 @@ extern "C" {
 
   VALUE rocksdb_iterator_seek(VALUE klass, VALUE v_target){
     Check_Type(v_target, T_STRING);
-    rocksdb::Slice target = rocksdb::Slice((char*)RSTRING_PTR(v_target));
+    rocksdb::Slice target = rocksdb::Slice((char*)RSTRING_PTR(v_target), RSTRING_LEN(v_target));
 
     rocksdb_iterator_pointer* rocksdb_it;
     Data_Get_Struct(klass, rocksdb_iterator_pointer , rocksdb_it);

data/lib/rocksdb/ruby/version.rb

@@ -1,5 +1,5 @@
 module Rocksdb
   module Ruby
-    VERSION = "0.1.3"
+    VERSION = "0.1.4"
   end
 end

data/spec/db_readonly_spec.rb

@@ -4,15 +4,19 @@ require "rocksdb"
 
 describe RocksDB do
   before do
-    @rocksdb = RocksDB::DB.new "/tmp/file2", {:readonly => true}
+    @rocksdb = RocksDB::DB.new "/tmp/file2"
+    @rocksdb.put("test:multi_db", "1")
+    @rocksdb.close
+
+    @rocksdb2 = RocksDB::DB.new "/tmp/file2", {:readonly => true}
   end
 
   it 'should get data' do
-    @rocksdb.put("test:multi_db", "10")
-    expect(@rocksdb.get("test:multi_db")).to eq "2"
+    @rocksdb2.put("test:multi_db", "10")
+    expect(@rocksdb2.get("test:multi_db")).to eq "1"
   end
 
   after do
-    @rocksdb.close
+    @rocksdb2.close
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rocksdb-ruby
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Isamu Arimoto