fixup

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

Author: joseph-isaacs

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

@@ -253,7 +253,7 @@ impl ExpressionView<'_, Binary> {
     }
 }
 
-/// Create a new [`Binary`] using the [`Eq`](crate::Operator::Eq) operator.
+/// Create a new [`Binary`] using the [`Eq`](crate::expr::exprs::operators::Operator::Eq) operator.
 ///
 /// ## Example usage
 ///
@@ -277,7 +277,7 @@ pub fn eq(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create Eq binary expression")
 }
 
-/// Create a new [`Binary`] using the [`NotEq`](crate::Operator::NotEq) operator.
+/// Create a new [`Binary`] using the [`NotEq`](crate::expr::exprs::operators::Operator::NotEq) operator.
 ///
 /// ## Example usage
 ///
@@ -301,7 +301,7 @@ pub fn not_eq(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create NotEq binary expression")
 }
 
-/// Create a new [`Binary`] using the [`Gte`](crate::Operator::Gte) operator.
+/// Create a new [`Binary`] using the [`Gte`](crate::expr::exprs::operators::Operator::Gte) operator.
 ///
 /// ## Example usage
 ///
@@ -325,7 +325,7 @@ pub fn gt_eq(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create Gte binary expression")
 }
 
-/// Create a new [`Binary`] using the [`Gt`](crate::Operator::Gt) operator.
+/// Create a new [`Binary`] using the [`Gt`](crate::expr::exprs::operators::Operator::Gt) operator.
 ///
 /// ## Example usage
 ///
@@ -349,7 +349,7 @@ pub fn gt(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create Gt binary expression")
 }
 
-/// Create a new [`Binary`] using the [`Lte`](crate::Operator::Lte) operator.
+/// Create a new [`Binary`] using the [`Lte`](crate::expr::exprs::operators::Operator::Lte) operator.
 ///
 /// ## Example usage
 ///
@@ -373,7 +373,7 @@ pub fn lt_eq(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create Lte binary expression")
 }
 
-/// Create a new [`Binary`] using the [`Lt`](crate::Operator::Lt) operator.
+/// Create a new [`Binary`] using the [`Lt`](crate::expr::exprs::operators::Operator::Lt) operator.
 ///
 /// ## Example usage
 ///
@@ -397,7 +397,7 @@ pub fn lt(lhs: Expression, rhs: Expression) -> Expression {
         .vortex_expect("Failed to create Lt binary expression")
 }
 
-/// Create a new [`Binary`] using the [`Or`](crate::Operator::Or) operator.
+/// Create a new [`Binary`] using the [`Or`](crate::expr::exprs::operators::Operator::Or) operator.
 ///
 /// ## Example usage
 ///
@@ -431,7 +431,7 @@ where
     Some(iter.rfold(first, |acc, elem| or(elem, acc)))
 }
 
-/// Create a new [`Binary`] using the [`And`](crate::Operator::And) operator.
+/// Create a new [`Binary`] using the [`And`](crate::expr::exprs::operators::Operator::And) operator.
 ///
 /// ## Example usage
 ///
@@ -475,7 +475,7 @@ where
     iter.reduce(and)
 }
 
-/// Create a new [`Binary`] using the [`Add`](crate::Operator::Add) operator.
+/// Create a new [`Binary`] using the [`Add`](crate::expr::exprs::operators::Operator::Add) operator.
 ///
 /// ## Example usage
 ///

vortex-array/src/expr/transform/mod.rs

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
-//! A collection of transformations that can be applied to a [`crate::Expression`].
+//! A collection of transformations that can be applied to a [`crate::expr::Expression`].
 pub mod annotations;
 pub mod immediate_access;
 pub(crate) mod match_between;

vortex-array/src/expr/vtable.rs

@@ -75,7 +75,7 @@ pub trait VTable: 'static + Sized + Send + Sync {
     /// Evaluate the expression in the given scope.
     fn evaluate(&self, expr: &ExpressionView<Self>, scope: &ArrayRef) -> VortexResult<ArrayRef>;
 
-    /// See [`crate::Expression::stat_falsification`].
+    /// See [`crate::expr::Expression::stat_falsification`].
     fn stat_falsification(
         &self,
         _expr: &ExpressionView<Self>,
@@ -84,7 +84,7 @@ pub trait VTable: 'static + Sized + Send + Sync {
         None
     }
 
-    /// See [`crate::Expression::stat_max`].
+    /// See [`crate::expr::Expression::stat_max`].
     fn stat_max(
         &self,
         _expr: &ExpressionView<Self>,
@@ -93,7 +93,7 @@ pub trait VTable: 'static + Sized + Send + Sync {
         None
     }
 
-    /// See [`crate::Expression::stat_min`].
+    /// See [`crate::expr::Expression::stat_min`].
     fn stat_min(
         &self,
         _expr: &ExpressionView<Self>,
@@ -102,7 +102,7 @@ pub trait VTable: 'static + Sized + Send + Sync {
         None
     }
 
-    /// See [`crate::Expression::stat_nan_count`].
+    /// See [`crate::expr::Expression::stat_nan_count`].
     fn stat_nan_count(
         &self,
         _expr: &ExpressionView<Self>,
@@ -111,7 +111,7 @@ pub trait VTable: 'static + Sized + Send + Sync {
         None
     }
 
-    /// See [`crate::Expression::stat_field_path`].
+    /// See [`crate::expr::Expression::stat_field_path`].
     fn stat_field_path(&self, _expr: &ExpressionView<Self>) -> Option<FieldPath> {
         None
     }