Add remaining scalar function validity hooks

Signed-off-by: Nicholas Gates <nick@nickgates.com>

Author: gatesn

vortex-geo/src/scalar_fn/distance.rs

@@ -13,6 +13,8 @@ use vortex_array::arrays::ScalarFnArray;
 use vortex_array::dtype::DType;
 use vortex_array::dtype::Nullability;
 use vortex_array::dtype::PType;
+use vortex_array::expr::Expression;
+use vortex_array::expr::lit;
 use vortex_array::scalar::Scalar;
 use vortex_array::scalar_fn::Arity;
 use vortex_array::scalar_fn::ChildName;
@@ -125,6 +127,14 @@ impl ScalarFnVTable for GeoDistance {
             }
         }
     }
+
+    fn validity(
+        &self,
+        _options: &Self::Options,
+        _expression: &Expression,
+    ) -> VortexResult<Expression> {
+        Ok(lit(true))
+    }
 }
 
 /// Distance from each row of `points` to a constant `query` point, decoded once and broadcast.

vortex-row/src/encode.rs

@@ -18,6 +18,8 @@ use vortex_array::arrays::PrimitiveArray;
 use vortex_array::dtype::DType;
 use vortex_array::dtype::Nullability;
 use vortex_array::dtype::PType;
+use vortex_array::expr::Expression;
+use vortex_array::expr::lit;
 use vortex_array::scalar_fn::Arity;
 use vortex_array::scalar_fn::ChildName;
 use vortex_array::scalar_fn::ExecutionArgs;
@@ -90,6 +92,14 @@ impl ScalarFnVTable for RowEncode {
         execute_row_encode(options, args, ctx)
     }
 
+    fn validity(
+        &self,
+        _options: &Self::Options,
+        _expression: &Expression,
+    ) -> VortexResult<Expression> {
+        Ok(lit(true))
+    }
+
     fn is_null_sensitive(&self, _options: &Self::Options) -> bool {
         true
     }

vortex-row/src/size.rs

@@ -18,6 +18,8 @@ use vortex_array::dtype::FieldNames;
 use vortex_array::dtype::Nullability;
 use vortex_array::dtype::PType;
 use vortex_array::dtype::StructFields;
+use vortex_array::expr::Expression;
+use vortex_array::expr::lit;
 use vortex_array::scalar::Scalar;
 use vortex_array::scalar_fn::Arity;
 use vortex_array::scalar_fn::ChildName;
@@ -242,6 +244,14 @@ impl ScalarFnVTable for RowSize {
         .into_array())
     }
 
+    fn validity(
+        &self,
+        _options: &Self::Options,
+        _expression: &Expression,
+    ) -> VortexResult<Expression> {
+        Ok(lit(true))
+    }
+
     fn is_null_sensitive(&self, _options: &Self::Options) -> bool {
         true
     }