Enable CI when PR targets non-develop branch

Signed-off-by: Nicholas Gates <[email protected]>

Author: gatesn

encodings/alp/src/alp/array.rs

@@ -475,14 +475,16 @@ mod tests {
     use rstest::rstest;
     use vortex_array::VectorExecutor;
     use vortex_array::arrays::PrimitiveArray;
+    use vortex_array::session::ArraySession;
     use vortex_array::vtable::ValidityHelper;
     use vortex_dtype::PTypeDowncast;
     use vortex_session::VortexSession;
     use vortex_vector::VectorOps;
 
     use super::*;
 
-    static SESSION: LazyLock<VortexSession> = LazyLock::new(VortexSession::empty);
+    static SESSION: LazyLock<VortexSession> =
+        LazyLock::new(|| VortexSession::empty().with::<ArraySession>());
 
     #[rstest]
     #[case(0)]

vortex-layout/src/layouts/flat/reader.rs

@@ -243,7 +243,6 @@ impl LayoutReader for FlatReader {
                 // Filter the array based on the row mask.
                 if !mask.all_true() {
                     array = filter(&array, &mask)?;
-                    array = array.filter(&mask)?;
                 }
 
                 // Evaluate the projection expression.

vortex-vector/src/bool/mod.rs

@@ -8,8 +8,8 @@
 //! ## Extending and appending
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! let mut vec1 = BoolVectorMut::from_iter([true, false].map(Some));
 //! let vec2 = BoolVectorMut::from_iter([true, true].map(Some)).freeze();
@@ -26,8 +26,8 @@
 //! ## Splitting and unsplitting
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! let mut vec = BoolVectorMut::from_iter([true, false, true, false, true].map(Some));
 //!
@@ -44,8 +44,8 @@
 //! ## Converting to immutable
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::{VectorMutOps, VectorOps};
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::{VectorMutOps, VectorOps};
 //!
 //! let mut vec = BoolVectorMut::from_iter([true, false, true].map(Some));
 //!

vortex-vector/src/decimal/mod.rs

@@ -15,9 +15,9 @@
 //! ## Creating and building decimal vectors
 //!
 //! ```
-//! use vortex_dtype::{PrecisionScale};
-//! use crate::decimal::{DVectorMut};
-//! use crate::VectorMutOps;
+//! use vortex_dtype::PrecisionScale;
+//! use vortex_vector::decimal::DVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! // Create a decimal vector with precision=9, scale=2 (e.g., up to 9999999.99).
 //! let ps = PrecisionScale::<i32>::new(9, 2);
@@ -48,10 +48,10 @@
 //!
 //! ```
 //! use vortex_buffer::BufferMut;
-//! use vortex_dtype::{PrecisionScale};
+//! use vortex_dtype::PrecisionScale;
 //! use vortex_mask::MaskMut;
-//! use crate::decimal::DVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::decimal::DVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! // Create a decimal vector with nulls.
 //! let ps = PrecisionScale::<i32>::new(5, 2); // Up to 999.99.
@@ -78,9 +78,9 @@
 //! ## Extending and manipulating vectors
 //!
 //! ```
-//! use vortex_dtype::{PrecisionScale};
-//! use crate::decimal::DVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_dtype::PrecisionScale;
+//! use vortex_vector::decimal::DVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! // Create two decimal vectors with scale=3 (3 decimal places).
 //! let ps = PrecisionScale::<i64>::new(10, 3);
@@ -114,9 +114,9 @@
 //! ## Converting between mutable and immutable
 //!
 //! ```
-//! use vortex_dtype::{PrecisionScale};
-//! use crate::decimal::DVectorMut;
-//! use crate::{VectorMutOps, VectorOps};
+//! use vortex_dtype::PrecisionScale;
+//! use vortex_vector::decimal::DVectorMut;
+//! use vortex_vector::{VectorMutOps, VectorOps};
 //!
 //! // Create a mutable decimal vector.
 //! let ps = PrecisionScale::<i128>::new(18, 6);  // High precision with 6 decimal places.

vortex-vector/src/fixed_size_list/mod.rs

@@ -11,9 +11,9 @@
 //! be null.
 //!
 //! ```
-//! use crate::fixed_size_list::FixedSizeListVectorMut;
-//! use crate::primitive::PVectorMut;
-//! use crate::{VectorMut, VectorMutOps};
+//! use vortex_vector::fixed_size_list::FixedSizeListVectorMut;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::{VectorMut, VectorMutOps};
 //! use vortex_mask::{Mask, MaskMut};
 //!
 //! // Create elements with some null values.
@@ -48,9 +48,9 @@
 //! [`unsplit()`]: crate::VectorMutOps::unsplit
 //!
 //! ```
-//! use crate::fixed_size_list::FixedSizeListVectorMut;
-//! use crate::primitive::PVectorMut;
-//! use crate::{VectorMut, VectorMutOps};
+//! use vortex_vector::fixed_size_list::FixedSizeListVectorMut;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::{VectorMut, VectorMutOps};
 //! use vortex_mask::MaskMut;
 //!
 //! // Create a vector with 6 lists, each containing 2 integers.

vortex-vector/src/primitive/mod.rs

@@ -16,8 +16,8 @@
 //! ## Creating and building a vector
 //!
 //! ```
-//! use crate::primitive::PVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! // Create with initial capacity for i32 values.
 //! let mut vec = PVectorMut::<i32>::with_capacity(10);
@@ -36,8 +36,8 @@
 //! ## Extending and appending
 //!
 //! ```
-//! use crate::primitive::PVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! let mut vec1 = PVectorMut::<i32>::from_iter([1, 2].map(Some));
 //! let vec2 = PVectorMut::<i32>::from_iter([3, 4].map(Some)).freeze();
@@ -54,8 +54,8 @@
 //! ## Splitting and unsplitting
 //!
 //! ```
-//! use crate::primitive::PVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! let mut vec = PVectorMut::<i64>::from_iter([10, 20, 30, 40, 50].map(Some));
 //!
@@ -72,8 +72,8 @@
 //! ## Working with nulls
 //!
 //! ```
-//! use crate::primitive::PVectorMut;
-//! use crate::VectorMutOps;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::VectorMutOps;
 //!
 //! // Create a vector with some null values.
 //! let mut vec = PVectorMut::<u32>::from_iter([Some(100), None, Some(200), None]);
@@ -87,8 +87,8 @@
 //! ## Converting to immutable
 //!
 //! ```
-//! use crate::primitive::PVectorMut;
-//! use crate::{VectorMutOps, VectorOps};
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::{VectorMutOps, VectorOps};
 //!
 //! let mut vec = PVectorMut::<f32>::from_iter([1.0, 2.0, 3.0].map(Some));
 //!

vortex-vector/src/struct_/mod.rs

@@ -8,11 +8,11 @@
 //! ## Creating a [`StructVector`] and [`StructVectorMut`]
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::null::NullVectorMut;
-//! use crate::primitive::PVectorMut;
-//! use crate::struct_::StructVectorMut;
-//! use crate::{VectorMut, VectorMutOps};
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::null::NullVectorMut;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::struct_::StructVectorMut;
+//! use vortex_vector::{VectorMut, VectorMutOps};
 //! use vortex_mask::MaskMut;
 //!
 //! // Create a struct with three fields: nulls, booleans, and integers.
@@ -32,11 +32,11 @@
 //! [`unsplit()`]: crate::VectorMutOps::unsplit
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::null::NullVectorMut;
-//! use crate::primitive::PVectorMut;
-//! use crate::struct_::StructVectorMut;
-//! use crate::{VectorMut, VectorMutOps};
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::null::NullVectorMut;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::struct_::StructVectorMut;
+//! use vortex_vector::{VectorMut, VectorMutOps};
 //! use vortex_mask::MaskMut;
 //!
 //! let fields = Box::new([
@@ -60,11 +60,11 @@
 //! ## Accessing field values
 //!
 //! ```
-//! use crate::bool::BoolVectorMut;
-//! use crate::null::NullVectorMut;
-//! use crate::primitive::PVectorMut;
-//! use crate::struct_::StructVectorMut;
-//! use crate::{VectorMut, VectorMutOps};
+//! use vortex_vector::bool::BoolVectorMut;
+//! use vortex_vector::null::NullVectorMut;
+//! use vortex_vector::primitive::PVectorMut;
+//! use vortex_vector::struct_::StructVectorMut;
+//! use vortex_vector::{VectorMut, VectorMutOps};
 //! use vortex_mask::MaskMut;
 //! use vortex_dtype::PTypeDowncast;
 //!