Add more arrow execution

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

Author: gatesn

vortex-array/src/executor.rs

@@ -33,6 +33,7 @@ impl VectorExecutor for ArrayRef {
         // NOTE(ngates): in the future we can choose a different mode of execution, or run
         // optimization here, etc.
         let kernel = self.bind_kernel(&mut ctx)?;
+        tracing::debug!("Executing kernel:\n{:#?}", kernel);
         let result = kernel.execute()?;
 
         vortex_ensure!(

vortex-array/src/expr/vtable.rs

@@ -177,16 +177,16 @@ pub trait VTable: 'static + Sized + Send + Sync {
     /// An expression is null-sensitive if it directly operates on null values,
     /// such as `is_null`. Most expressions are not null-sensitive.
     ///
-    /// The property we are interested in is if the expression (e) distributes over
-    /// mask.
+    /// The property we are interested in is if the expression (e) distributes over `mask`.
     /// Define a `mask(a, m)` expression that applies the boolean array `m` to the validity of the
     /// array `a`.
-    /// An unary expression `e` to be null-sensitive iff forall arrays `a` and masks `m`.
+    ///
+    /// A unary expression `e` is not null-sensitive iff forall arrays `a` and masks `m`,
     /// `e(mask(a, m)) == mask(e(a), m)`.
+    ///
     /// This can be extended to an n-ary expression.
     ///
-    /// This method only checks the expression itself, not its children. To check
-    /// if an expression or any of its descendants are null-sensitive.
+    /// This method only checks the expression itself, not its children.
     fn is_null_sensitive(&self, options: &Self::Options) -> bool {
         _ = options;
         true