This is an automated email from the ASF dual-hosted git repository.
jeffreyvo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new d987d2dc80 minor: remove unnecessary clippy:large_enum_variant allows
(#17108)
d987d2dc80 is described below
commit d987d2dc80c702c86d7ac853fb9c251a291d7b91
Author: Jeffrey Vo <jeffrey.vo.austra...@gmail.com>
AuthorDate: Mon Aug 11 14:57:36 2025 +1000
minor: remove unnecessary clippy:large_enum_variant allows (#17108)
---
datafusion/expr/src/planner.rs | 1 -
datafusion/expr/src/simplify.rs | 1 -
datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs | 1 -
3 files changed, 3 deletions(-)
diff --git a/datafusion/expr/src/planner.rs b/datafusion/expr/src/planner.rs
index b04fe32d37..24d1993956 100644
--- a/datafusion/expr/src/planner.rs
+++ b/datafusion/expr/src/planner.rs
@@ -313,7 +313,6 @@ pub struct RawWindowExpr {
/// Result of planning a raw expr with [`ExprPlanner`]
#[derive(Debug, Clone)]
-#[allow(clippy::large_enum_variant)]
pub enum PlannerResult<T> {
/// The raw expression was successfully planned as a new [`Expr`]
Planned(Expr),
diff --git a/datafusion/expr/src/simplify.rs b/datafusion/expr/src/simplify.rs
index 411dbbdc40..467ce8bf53 100644
--- a/datafusion/expr/src/simplify.rs
+++ b/datafusion/expr/src/simplify.rs
@@ -110,7 +110,6 @@ impl SimplifyInfo for SimplifyContext<'_> {
/// Was the expression simplified?
#[derive(Debug)]
-#[allow(clippy::large_enum_variant)]
pub enum ExprSimplifyResult {
/// The function call was simplified to an entirely new Expr
Simplified(Expr),
diff --git a/datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs
b/datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs
index d33cefb341..4f63697ff9 100644
--- a/datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs
+++ b/datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs
@@ -519,7 +519,6 @@ struct ConstEvaluator<'a> {
#[allow(dead_code)]
/// The simplify result of ConstEvaluator
-#[allow(clippy::large_enum_variant)]
enum ConstSimplifyResult {
// Expr was simplified and contains the new expression
Simplified(ScalarValue, Option<FieldMetadata>),
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org
