finish

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>

Author: joseph-isaacs

vortex-array/src/expr/exprs/mask.rs

@@ -2,6 +2,7 @@
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
 use std::fmt::Formatter;
+use std::ops::Not;
 
 use vortex_dtype::DType;
 use vortex_dtype::Nullability;

vortex-layout/src/layouts/row_idx/expr.rs

@@ -3,7 +3,6 @@
 
 use std::fmt::Formatter;
 
-use vortex_array::ArrayRef;
 use vortex_array::expr::Arity;
 use vortex_array::expr::ChildName;
 use vortex_array::expr::EmptyOptions;
@@ -49,17 +48,6 @@ impl VTable for RowIdx {
         Ok(DType::Primitive(PType::U64, Nullability::NonNullable))
     }
 
-    fn evaluate(
-        &self,
-        _options: &Self::Options,
-        _expr: &Expression,
-        _scope: &ArrayRef,
-    ) -> VortexResult<ArrayRef> {
-        vortex_bail!(
-            "RowIdxExpr should not be evaluated directly, use it in the context of a Vortex scan and it will be substituted for a row index array"
-        );
-    }
-
     fn execute(
         &self,
         _options: &Self::Options,