Skip to content

Commit

Permalink
Remove deprecated LogicalPlan::inspect_expressions
Browse files Browse the repository at this point in the history
Deprecated since v 37.
  • Loading branch information
findepi committed Oct 22, 2024
1 parent 64758e8 commit 46dd49a
Showing 1 changed file with 0 additions and 21 deletions.
21 changes: 0 additions & 21 deletions datafusion/expr/src/logical_plan/plan.rs
Original file line number Diff line number Diff line change
Expand Up @@ -423,27 +423,6 @@ impl LogicalPlan {
exprs
}

#[deprecated(since = "37.0.0", note = "Use `apply_expressions` instead")]
pub fn inspect_expressions<F, E>(self: &LogicalPlan, mut f: F) -> Result<(), E>
where
F: FnMut(&Expr) -> Result<(), E>,
{
let mut err = Ok(());
self.apply_expressions(|e| {
if let Err(e) = f(e) {
// save the error for later (it may not be a DataFusionError
err = Err(e);
Ok(TreeNodeRecursion::Stop)
} else {
Ok(TreeNodeRecursion::Continue)
}
})
// The closure always returns OK, so this will always too
.expect("no way to return error during recursion");

err
}

/// Returns all inputs / children of this `LogicalPlan` node.
///
/// Note does not include inputs to inputs, or subqueries.
Expand Down

0 comments on commit 46dd49a

Please sign in to comment.