parquet 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 259f41f1ea1b111a0b0fdec15d17b54ac5a4efee750157159a33a4272b0b1310
-  data.tar.gz: 71579d25b3ec411208103e4afe116285bd75a30003af3f21b7d7972ef8942ff6
+  metadata.gz: b44b6f3ca56a0f4318c361b309c1af213186dcf09e09acbe54561a2dcc920042
+  data.tar.gz: 68aa0636e1467e008ec29bbdb43737904b8f0a26f2d91a5169005f896950687f
 SHA512:
-  metadata.gz: 792a6653554393a94de0572ad2164e5da89c7b49b9476599b9d45efd38448f6e7065fa5b7b0082e036ea19da19441b462083ca9e61226f82dd38a812905a189e
-  data.tar.gz: 1520073d0668751a5c449dde08deea5dd06e50845e8e8fead398d7015d5cc979885897a258f1ab86eb405203fcaf6ebe0164ccdb31e4033e4911ad4b68268d11
+  metadata.gz: 30966ae0335b9caa4458a79e5d627b920f2368605785a978690490a7ed05beb1f7ab25eeeed7349cd3c05b49573dc68405cb8c52af5aad5240a42ce4a56b184b
+  data.tar.gz: 5ea6021844baac3bb31acf41f0fdc3795710f9eff21e9c3556692e70f10398249d802b6f3bb22f414d282f2ff33a2ea951fe9d3040625002752d86311eeae160

data/ext/parquet/src/ruby_reader.rs

@@ -34,9 +34,10 @@ impl RubyReader {
     }
 
     // For now, don't use this. Having to use seek in length is scary.
-    fn is_seekable_io_like(_value: &Value) -> bool {
-        // Self::is_io_like(value) && value.respond_to("seek", false).unwrap_or(false)
-        false
+    fn is_seekable_io_like(value: &Value) -> bool {
+        Self::is_io_like(value)
+            && value.respond_to("seek", false).unwrap_or(false)
+            && value.respond_to("pos", false).unwrap_or(false)
     }
 }
 
@@ -92,10 +93,14 @@ impl Seek for RubyReader {
                 let unwrapped_inner = ruby.get_inner(*inner);
 
                 let new_offset = match pos {
-                    io::SeekFrom::Start(offset) => offset as usize,
-                    io::SeekFrom::Current(offset) => (*original_offset as i64 + offset) as usize,
-                    io::SeekFrom::End(offset) => {
-                        unwrapped_inner.len().saturating_sub(offset as usize)
+                    SeekFrom::Start(off) => off as usize,
+                    SeekFrom::Current(off) => {
+                        let signed = *original_offset as i64 + off;
+                        signed.max(0) as usize
+                    }
+                    SeekFrom::End(off) => {
+                        let signed = unwrapped_inner.len() as i64 + off;
+                        signed.max(0) as usize
                     }
                 };
 
@@ -111,8 +116,12 @@ impl Seek for RubyReader {
                     SeekFrom::End(i) => (2, i),
                 };
 
+                unwrapped_inner
+                    .funcall::<_, _, u64>("seek", (ruby_offset, whence))
+                    .map_err(|e| io::Error::new(io::ErrorKind::Other, e.to_string()))?;
+
                 let new_position = unwrapped_inner
-                    .funcall("seek", (ruby_offset, whence))
+                    .funcall::<_, _, u64>("pos", ())
                     .map_err(|e| io::Error::new(io::ErrorKind::Other, e.to_string()))?;
 
                 Ok(new_position)

data/lib/parquet/version.rb

@@ -1,3 +1,3 @@
 module Parquet
-  VERSION = "0.3.2"
+  VERSION = "0.3.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: parquet
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Nathan Jaremko