Chore: Reorganize documentation and modules for vectors (#5186)

Tracking Issue: #5028

Now that we have basically all of the vectors (`List`s incoming later
today), the crate namespace is getting super crowded. This PR removes
the re-exporting into the crate level and makes all vectors accessible
via the vortex-vector submodules. Additionally, this moves the main
examples for each vector type into their respective modules (rather than
on the mutable vector type directly).

This is a **purely cosmetic change** that makes it easier to navigate
the documentation of `vortex-vector`. Given that this is effectively
going to be Vortex's IR, I think it is important to be more careful
about what we put into the top-level namespace.

If people don't like having to type out an extra word when importing,
then I would suggest we add a `prelude` rather than forcing everything
at the top level.

---------

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>

Author: connortsui20

encodings/sequence/src/operator.rs

@@ -10,7 +10,8 @@ use vortex_array::vtable::OperatorVTable;
 use vortex_dtype::{NativePType, match_each_native_ptype};
 use vortex_error::{VortexExpect, VortexResult};
 use vortex_mask::AllOr;
-use vortex_vector::{PVectorMut, Vector, VectorMutOps};
+use vortex_vector::primitive::PVectorMut;
+use vortex_vector::{Vector, VectorMutOps};
 
 use crate::{SequenceArray, SequenceVTable};
 

vortex-array/src/arrays/bool/vtable/operator.rs

@@ -3,7 +3,7 @@
 
 use vortex_compute::filter::Filter;
 use vortex_error::VortexResult;
-use vortex_vector::BoolVector;
+use vortex_vector::bool::BoolVector;
 
 use crate::arrays::{BoolArray, BoolVTable, MaskedVTable};
 use crate::execution::{BatchKernelRef, BindCtx, kernel};

vortex-array/src/arrays/null/mod.rs

@@ -9,7 +9,7 @@ use vortex_dtype::DType;
 use vortex_error::VortexResult;
 use vortex_mask::Mask;
 use vortex_scalar::Scalar;
-use vortex_vector::NullVector;
+use vortex_vector::null::NullVector;
 
 use crate::execution::{BatchKernelRef, BindCtx, kernel};
 use crate::serde::ArrayChildren;

vortex-array/src/arrays/primitive/array/conversion.rs

@@ -6,7 +6,8 @@
 use vortex_buffer::{BitBufferMut, Buffer, BufferMut};
 use vortex_dtype::{NativePType, Nullability};
 use vortex_error::{VortexResult, vortex_ensure, vortex_panic};
-use vortex_vector::{PrimitiveVector, VectorOps, match_each_pvector};
+use vortex_vector::primitive::PrimitiveVector;
+use vortex_vector::{VectorOps, match_each_pvector};
 
 use crate::arrays::PrimitiveArray;
 use crate::validity::Validity;
@@ -142,7 +143,7 @@ mod tests {
     use vortex_buffer::BufferMut;
     use vortex_dtype::{Nullability, PType};
     use vortex_mask::MaskMut;
-    use vortex_vector::PVector;
+    use vortex_vector::primitive::PVector;
 
     use super::*;
 

vortex-array/src/arrays/primitive/vtable/operator.rs

@@ -4,7 +4,7 @@
 use vortex_compute::filter::Filter;
 use vortex_dtype::match_each_native_ptype;
 use vortex_error::VortexResult;
-use vortex_vector::PVector;
+use vortex_vector::primitive::PVector;
 
 use crate::arrays::{MaskedVTable, PrimitiveArray, PrimitiveVTable};
 use crate::execution::{BatchKernelRef, BindCtx, kernel};

vortex-array/src/arrays/struct_/vtable/operator.rs

@@ -4,7 +4,8 @@
 use std::sync::Arc;
 
 use vortex_error::VortexResult;
-use vortex_vector::{StructVector, Vector};
+use vortex_vector::Vector;
+use vortex_vector::struct_::StructVector;
 
 use crate::ArrayRef;
 use crate::arrays::{StructArray, StructVTable};

vortex-array/src/compute/arrays/arithmetic.rs

@@ -12,7 +12,7 @@ use vortex_compute::arithmetic::{
 use vortex_dtype::{DType, NativePType, PTypeDowncastExt, match_each_native_ptype};
 use vortex_error::{VortexExpect, VortexResult, vortex_err};
 use vortex_scalar::{PValue, Scalar};
-use vortex_vector::PVector;
+use vortex_vector::primitive::PVector;
 
 use crate::arrays::ConstantArray;
 use crate::execution::{BatchKernelRef, BindCtx, kernel};

vortex-array/src/compute/arrays/is_not_null.rs

@@ -7,7 +7,8 @@ use vortex_dtype::DType;
 use vortex_dtype::Nullability::NonNullable;
 use vortex_error::VortexResult;
 use vortex_mask::Mask;
-use vortex_vector::{BoolVector, VectorOps};
+use vortex_vector::VectorOps;
+use vortex_vector::bool::BoolVector;
 
 use crate::execution::{BatchKernelRef, BindCtx, kernel};
 use crate::stats::{ArrayStats, StatsSetRef};

vortex-array/src/compute/arrays/is_null.rs

@@ -8,7 +8,8 @@ use vortex_dtype::DType;
 use vortex_dtype::Nullability::NonNullable;
 use vortex_error::VortexResult;
 use vortex_mask::Mask;
-use vortex_vector::{BoolVector, VectorOps};
+use vortex_vector::VectorOps;
+use vortex_vector::bool::BoolVector;
 
 use crate::execution::{BatchKernelRef, BindCtx, kernel};
 use crate::stats::{ArrayStats, StatsSetRef};

vortex-array/src/compute/arrays/logical.rs

@@ -11,7 +11,7 @@ use vortex_compute::logical::{
 };
 use vortex_dtype::DType;
 use vortex_error::VortexResult;
-use vortex_vector::BoolVector;
+use vortex_vector::bool::BoolVector;
 
 use crate::execution::{BatchKernelRef, BindCtx, kernel};
 use crate::serde::ArrayChildren;