feat[vortex-array]: add an expr array

Signed-off-by: Joe Isaacs <[email protected]>

Author: joseph-isaacs

vortex-array/src/arrays/expr/vtable/array.rs

@@ -28,7 +28,12 @@ impl ArrayVTable<ExprVTable> for ExprVTable {
         array.child.array_hash(state, precision);
         array.dtype.hash(state);
         // TODO(joe): fixme
-        array.expr.serialize_metadata().hash()
+        array
+            .expr
+            .serialize_metadata()
+            .unwrap_or(None)
+            .unwrap_or(vec![])
+            .hash(state)
         // Note: Expression doesn't implement Hash, so we skip it
         // This is acceptable since expressions are typically transient
     }
@@ -37,6 +42,6 @@ impl ArrayVTable<ExprVTable> for ExprVTable {
         array.child.array_eq(&other.child, precision)
             && array.dtype == other.dtype
             // TODO(joe): fixme
-            && array.expr.serialize_metadata() == other.expr.serialize_metadata()
+            && array.expr.serialize_metadata().unwrap_or(None) == other.expr.serialize_metadata().unwrap_or(None)
     }
 }

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

@@ -28,7 +28,7 @@ mod tests {
     use crate::expr::binary::checked_add;
     use crate::expr::literal::lit;
     use crate::validity::Validity;
-    use crate::{Array, IntoArray};
+    use crate::{Array, IntoArray, assert_arrays_eq};
 
     #[test]
     fn test_expr_array_canonicalize() {
@@ -43,6 +43,6 @@ mod tests {
         let actual = expr_array.to_canonical().into_array();
 
         let expect = (0..3).map(|_| 15i32).collect::<PrimitiveArray>();
-        assert_eq!(expect, actual);
+        assert_arrays_eq!(expect, actual);
     }
 }

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

@@ -5,7 +5,6 @@ use vortex_error::VortexResult;
 
 use crate::ArrayRef;
 use crate::arrays::expr::{ExprArray, ExprVTable};
-use crate::execution::{BatchKernelRef, BindCtx};
 use crate::expr::root;
 use crate::expr::session::ExprSession;
 use crate::expr::transform::ExprOptimizer;