fix clippy

apache · Nov 15, 2024 · 8630128 · 8630128
1 parent e7ec973
commit 8630128
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git 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
@@ -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();