Use generic TakeSlices path for ListView rebuild

Signed-off-by: Daniel King <dan@spiraldb.com>

Author: danking

vortex-array/src/arrays/listview/rebuild.rs

@@ -15,10 +15,8 @@ use crate::arrays::ConstantArray;
 use crate::arrays::ListViewArray;
 use crate::arrays::PrimitiveArray;
 use crate::arrays::TakeSlicesArray;
-use crate::arrays::VarBinView;
 use crate::arrays::listview::ListViewArrayExt;
 use crate::arrays::primitive::PrimitiveArrayExt;
-use crate::arrays::take_slices::TakeSlicesKernel;
 use crate::builtins::ArrayBuiltins;
 use crate::dtype::IntegerPType;
 use crate::dtype::Nullability;
@@ -297,22 +295,12 @@ impl ListViewArray {
         let starts = PrimitiveArray::new(starts.freeze(), Validity::NonNullable).into_array();
         let lengths = PrimitiveArray::new(lengths.freeze(), Validity::NonNullable).into_array();
         let new_sizes = new_sizes.freeze();
-        let elements = if let Some(elements) = self.elements().as_opt::<VarBinView>() {
-            <VarBinView as TakeSlicesKernel>::take_slices(
-                elements,
-                &starts,
-                &lengths,
-                elements_len,
-                ctx,
-            )?
-            .vortex_expect("VarBinView TakeSlicesKernel should produce an array")
-        } else {
+        let elements =
             TakeSlicesArray::try_new(self.elements().clone(), starts, lengths, elements_len)?
                 .into_array()
                 .execute::<RecursiveCanonical>(ctx)?
                 .0
-                .into_array()
-        };
+                .into_array();
         // Built unsigned; reinterpret back to the signed-preserving result types.
         let offsets = PrimitiveArray::new(new_offsets.freeze(), Validity::NonNullable)
             .reinterpret_cast(new_offset_ptype)