diff --git a/datafusion/functions-nested/src/planner.rs b/datafusion/functions-nested/src/planner.rs index 82787a65d506..1929b8222a1b 100644 --- a/datafusion/functions-nested/src/planner.rs +++ b/datafusion/functions-nested/src/planner.rs @@ -34,7 +34,7 @@ use crate::{ make_array::make_array, }; -#[derive(Default, Debug)] +#[derive(Debug)] pub struct NestedFunctionPlanner; impl ExprPlanner for NestedFunctionPlanner { @@ -131,9 +131,8 @@ impl ExprPlanner for NestedFunctionPlanner { } } -#[derive(Default, Debug)] +#[derive(Debug)] pub struct FieldAccessPlanner; - impl ExprPlanner for FieldAccessPlanner { fn plan_field_access( &self, diff --git a/datafusion/sql/tests/cases/plan_to_sql.rs b/datafusion/sql/tests/cases/plan_to_sql.rs index 4b84395f197e..4f43d7333dd1 100644 --- a/datafusion/sql/tests/cases/plan_to_sql.rs +++ b/datafusion/sql/tests/cases/plan_to_sql.rs @@ -208,8 +208,8 @@ fn roundtrip_statement() -> Result<()> { .with_aggregate_function(count_udaf()) .with_aggregate_function(max_udaf()) .with_expr_planner(Arc::new(CoreFunctionPlanner::default())) - .with_expr_planner(Arc::new(NestedFunctionPlanner::default())) - .with_expr_planner(Arc::new(FieldAccessPlanner::default())); + .with_expr_planner(Arc::new(NestedFunctionPlanner)) + .with_expr_planner(Arc::new(FieldAccessPlanner)); let context = MockContextProvider { state }; let sql_to_rel = SqlToRel::new(&context); let plan = sql_to_rel.sql_statement_to_plan(statement).unwrap();