simpler

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

Author: robert3005

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

@@ -386,16 +386,16 @@ mod tests {
         let clobbered_indices =
             PrimitiveArray::new(Buffer::from(indices_data), indices_prim.validity()).into_array();
 
-        Ok(unsafe {
-            RLEArra::new_unchecked(
+        unsafe {
+            RLEArray::try_from_data(RLEData::new_unchecked(
                 rle.values().clone(),
                 clobbered_indices,
                 rle.values_idx_offsets().clone(),
                 rle.dtype().clone(),
                 rle.offset(),
                 rle.len(),
-            )
-        })
+            ))
+        }
     }
 
     #[test]

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

@@ -3,6 +3,7 @@
 
 use fastlanes::RLE;
 use num_traits::AsPrimitive;
+use num_traits::NumCast;
 use vortex_array::ExecutionCtx;
 use vortex_array::IntoArray;
 use vortex_array::arrays::PrimitiveArray;
@@ -24,10 +25,6 @@ use crate::rle::RLEArrayExt;
     reason = "complexity is from nested match_each_* macros"
 )]
 pub fn rle_decompress(array: &RLEArray, ctx: &mut ExecutionCtx) -> VortexResult<PrimitiveArray> {
-    if array.all_invalid()? {
-        return Ok(Canonical::empty(array.dtype()).into_primitive());
-    }
-
     match_each_native_ptype!(array.values().dtype().as_ptype(), |V| {
         match_each_unsigned_integer_ptype!(array.values_idx_offsets().dtype().as_ptype(), |O| {
             // RLE indices are always u16 (or u8 if downcasted).