Remove ExprArray and ExprOptimizer (#5585)

In favor of using ScalarFnArray in the future

---------

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

Author: gatesn

encodings/pco/src/test.rs

@@ -3,7 +3,6 @@
 #![allow(clippy::cast_possible_truncation)]
 
 use vortex_array::ArrayContext;
-use vortex_array::ArraySession;
 use vortex_array::IntoArray;
 use vortex_array::ToCanonical;
 use vortex_array::arrays::BoolArray;
@@ -12,6 +11,7 @@ use vortex_array::arrow::compute::to_arrow_preferred;
 use vortex_array::assert_arrays_eq;
 use vortex_array::serde::ArrayParts;
 use vortex_array::serde::SerializeOptions;
+use vortex_array::session::ArraySession;
 use vortex_array::validity::Validity;
 use vortex_array::vtable::ArrayVTableExt;
 use vortex_array::vtable::ValidityHelper;

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

@@ -14,8 +14,7 @@ use crate::arrays::MaskedVTable;
 use crate::execution::BatchKernelRef;
 use crate::execution::BindCtx;
 use crate::execution::kernel;
-use crate::transform::ArrayParentReduceRule;
-use crate::transform::ArrayRuleContext;
+use crate::optimizer::rules::ArrayParentReduceRule;
 use crate::vtable::OperatorVTable;
 use crate::vtable::ValidityHelper;
 
@@ -54,7 +53,6 @@ impl ArrayParentReduceRule<BoolVTable, MaskedVTable> for BoolMaskedValidityRule
         array: &BoolArray,
         parent: &MaskedArray,
         _child_idx: usize,
-        _ctx: &ArrayRuleContext,
     ) -> VortexResult<Option<ArrayRef>> {
         // Merge the parent's validity mask into the child's validity
         // TODO(joe): make this lazy

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

@@ -16,8 +16,7 @@ use crate::arrays::MaskedVTable;
 use crate::execution::BatchKernelRef;
 use crate::execution::BindCtx;
 use crate::execution::kernel;
-use crate::transform::ArrayParentReduceRule;
-use crate::transform::ArrayRuleContext;
+use crate::optimizer::rules::ArrayParentReduceRule;
 use crate::vtable::OperatorVTable;
 use crate::vtable::ValidityHelper;
 
@@ -61,7 +60,6 @@ impl ArrayParentReduceRule<DecimalVTable, MaskedVTable> for DecimalMaskedValidit
         array: &DecimalArray,
         parent: &MaskedArray,
         _child_idx: usize,
-        _ctx: &ArrayRuleContext,
     ) -> VortexResult<Option<ArrayRef>> {
         // Merge the parent's validity mask into the child's validity
         // TODO(joe): make this lazy