less

Signed-off-by: Robert Kruszewski <github@robertk.io>

Author: robert3005

encodings/fastlanes/src/bitpacking/vtable/mod.rs

@@ -252,18 +252,6 @@ impl VTable for BitPacked {
     }
 
     fn serialize(metadata: Self::Metadata) -> VortexResult<Option<Vec<u8>>> {
-        vortex_ensure!(
-            metadata.offset == 0,
-            "BitPackedArray offset must be 0 for serialization, got {}",
-            metadata.offset
-        );
-        if let Some(ref patches) = metadata.patches {
-            vortex_ensure!(
-                patches.offset()? == 0,
-                "BitPackedArray patches offset must be 0 for serialization, got {}",
-                patches.offset()?
-            );
-        }
         Ok(Some(metadata.serialize()))
     }
 

encodings/fastlanes/src/delta/vtable/mod.rs

@@ -157,11 +157,6 @@ impl VTable for Delta {
     }
 
     fn serialize(metadata: Self::Metadata) -> VortexResult<Option<Vec<u8>>> {
-        vortex_ensure!(
-            metadata.offset == 0,
-            "DeltaArray offset must be 0 for serialization, got {}",
-            metadata.offset
-        );
         Ok(Some(metadata.0.encode_to_vec()))
     }
 

encodings/fastlanes/src/rle/array/mod.rs

@@ -504,16 +504,18 @@ mod tests {
         let concat = concat.freeze();
 
         let parts = ArrayParts::try_from(concat).unwrap();
-        let decoded = parts.decode(
-            sliced.dtype(),
-            sliced.len(),
-            &ReadContext::new(ctx.to_ids()),
-            &SESSION,
-        );
+        let decoded = parts
+            .decode(
+                sliced.dtype(),
+                sliced.len(),
+                &ReadContext::new(ctx.to_ids()),
+                &SESSION,
+            )
+            .unwrap();
 
-        assert_eq!(
-            decoded.err().unwrap().to_string(),
-            "Other error: RLEArray offset must be 0 for serialization, got 100"
-        );
+        let original_data = sliced.to_primitive();
+        let decoded_data = decoded.to_primitive();
+
+        assert_arrays_eq!(original_data, decoded_data);
     }
 }

encodings/fastlanes/src/rle/vtable/mod.rs

@@ -176,11 +176,6 @@ impl VTable for RLE {
     }
 
     fn serialize(metadata: Self::Metadata) -> VortexResult<Option<Vec<u8>>> {
-        vortex_ensure!(
-            metadata.offset == 0,
-            "RLEArray offset must be 0 for serialization, got {}",
-            metadata.offset
-        );
         Ok(Some(metadata.0.encode_to_vec()))
     }
 

encodings/runend/src/array.rs

@@ -142,11 +142,6 @@ impl VTable for RunEnd {
     }
 
     fn serialize(metadata: Self::Metadata) -> VortexResult<Option<Vec<u8>>> {
-        vortex_ensure!(
-            metadata.offset == 0,
-            "RunEndArray offset must be 0 for serialization, got {}",
-            metadata.offset
-        );
         Ok(Some(metadata.serialize()))
     }