chore[array]: remove vortex-vector dependency

vortex-array/Cargo.toml

@@ -53,6 +53,7 @@ rstest_reuse = { workspace = true, optional = true }
 rustc-hash = { workspace = true }
 serde = { workspace = true, optional = true, features = ["derive"] }
 simdutf8 = { workspace = true }
+static_assertions = { workspace = true }
 tabled = { workspace = true, optional = true, default-features = false, features = [
     "std",
 ] }
@@ -68,7 +69,6 @@ vortex-proto = { workspace = true, features = ["expr"] }
 vortex-scalar = { workspace = true }
 vortex-session = { workspace = true }
 vortex-utils = { workspace = true, features = ["dyn-traits"] }
-vortex-vector = { workspace = true }
 
 [features]
 arbitrary = [

vortex-array/benches/take_primitive.rs

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
-//! Benchmarks comparing [`PVector`] take vs [`DictArray`] canonicalization.
+//! Benchmarks for [`DictArray`] canonicalization.
 //!
 //! Both are tracked by number of indices/codes for fair comparison.
 
@@ -15,10 +15,6 @@ use rand_distr::Zipf;
 use vortex_array::IntoArray;
 use vortex_array::arrays::DictArray;
 use vortex_array::arrays::PrimitiveArray;
-use vortex_buffer::Buffer;
-use vortex_compute::take::Take;
-use vortex_mask::Mask;
-use vortex_vector::primitive::PVector;
 
 fn main() {
     divan::main();
@@ -30,40 +26,6 @@ const NUM_INDICES: &[usize] = &[1_000, 10_000, 100_000];
 /// Size of the source vector / dictionary values.
 const VECTOR_SIZE: &[usize] = &[16, 256, 2048, 8192];
 
-// --- PVector take benchmarks ---
-
-#[divan::bench(args = NUM_INDICES, consts = VECTOR_SIZE, sample_count = 100_000)]
-fn pvector_take_uniform<const VECTOR_SIZE: usize>(bencher: Bencher, num_indices: usize) {
-    let data: Buffer<u32> = (0..VECTOR_SIZE as u32).collect();
-    let pvector = PVector::new(data, Mask::AllTrue(VECTOR_SIZE));
-
-    let rng = StdRng::seed_from_u64(0);
-    let range = Uniform::new(0u32, VECTOR_SIZE as u32).unwrap();
-    let indices: Vec<u32> = rng.sample_iter(range).take(num_indices).collect();
-
-    bencher
-        .with_inputs(|| (&pvector, indices.as_slice()))
-        .bench_refs(|(pv, idx)| pv.take(*idx));
-}
-
-#[divan::bench(args = NUM_INDICES, consts = VECTOR_SIZE, sample_count = 100_000)]
-fn pvector_take_zipfian<const VECTOR_SIZE: usize>(bencher: Bencher, num_indices: usize) {
-    let data: Buffer<u32> = (0..VECTOR_SIZE as u32).collect();
-    let pvector = PVector::new(data, Mask::AllTrue(VECTOR_SIZE));
-
-    let rng = StdRng::seed_from_u64(0);
-    let zipf = Zipf::new(VECTOR_SIZE as f64, 1.0).unwrap();
-    let indices: Vec<u32> = rng
-        .sample_iter(&zipf)
-        .take(num_indices)
-        .map(|i: f64| (i as u32 - 1).min(VECTOR_SIZE as u32 - 1))
-        .collect();
-
-    bencher
-        .with_inputs(|| (&pvector, indices.as_slice()))
-        .bench_refs(|(pv, idx)| pv.take(*idx));
-}
-
 // --- DictArray canonicalization benchmarks ---
 
 #[divan::bench(args = NUM_INDICES, consts = VECTOR_SIZE, sample_count = 100_000)]

vortex-array/src/arrays/constant/vtable/canonical.rs

@@ -22,7 +22,6 @@ use vortex_scalar::ListScalar;
 use vortex_scalar::Scalar;
 use vortex_scalar::StructScalar;
 use vortex_scalar::Utf8Scalar;
-use vortex_vector::binaryview::BinaryView;
 
 use crate::Canonical;
 use crate::IntoArray;
@@ -34,6 +33,7 @@ use crate::arrays::ListViewArray;
 use crate::arrays::NullArray;
 use crate::arrays::StructArray;
 use crate::arrays::VarBinViewArray;
+use crate::arrays::build_views::BinaryView;
 use crate::arrays::constant::ConstantArray;
 use crate::arrays::primitive::PrimitiveArray;
 use crate::builders::builder_with_capacity;

vortex-array/src/arrays/list/compute/filter.rs

@@ -138,10 +138,8 @@ fn compute_filtered_elements_and_offsets<O: IntegerPType>(
 }
 
 /// Construct an element mask from contiguous list offsets and a selection mask.
-pub fn element_mask_from_offsets<O: IntegerPType>(
-    offsets: &[O],
-    selection: &Arc<MaskValues>,
-) -> Mask {
+#[allow(dead_code)]
+fn element_mask_from_offsets<O: IntegerPType>(offsets: &[O], selection: &Arc<MaskValues>) -> Mask {
     let first_offset = offsets.first().map_or(0, |first_offset| first_offset.as_());
     let last_offset = offsets.last().map_or(0, |last_offset| last_offset.as_());
     let len = last_offset - first_offset;

vortex-array/src/arrays/list/compute/mod.rs

@@ -9,8 +9,6 @@ mod mask;
 mod min_max;
 mod take;
 
-pub(super) use filter::element_mask_from_offsets;
-
 #[cfg(test)]
 mod tests {
     use rstest::rstest;

vortex-array/src/arrays/list/vtable/kernel/mod.rs

@@ -1,11 +1,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
-mod filter;
-
 use crate::arrays::ListVTable;
-use crate::arrays::list::vtable::kernel::filter::ListFilterKernel;
 use crate::kernel::ParentKernelSet;
 
-pub(super) const PARENT_KERNELS: ParentKernelSet<ListVTable> =
-    ParentKernelSet::new(&[ParentKernelSet::lift(&ListFilterKernel)]);
+pub(super) const PARENT_KERNELS: ParentKernelSet<ListVTable> = ParentKernelSet::new(&[]);

vortex-array/src/arrays/mod.rs

@@ -58,4 +58,6 @@ pub use scalar_fn::*;
 pub use slice::*;
 pub use struct_::*;
 pub use varbin::*;
+pub use varbinview::build_views;
+pub use varbinview::compact;
 pub use varbinview::*;