Removes unnecessary case expression for handling null values

apache · Dec 11, 2024 · 4668bf9 · 4668bf9
1 parent edcc168
commit 4668bf9
Showing 1 changed file with 1 addition and 11 deletions.
diff --git a/native/core/src/execution/datafusion/planner.rs b/native/core/src/execution/datafusion/planner.rs
@@ -708,17 +708,7 @@ impl PhysicalPlanner {
                     args,
                     return_type,
                 ));
-
-                let is_null_expr: Arc<dyn PhysicalExpr> = Arc::new(IsNullExpr::new(left));
-                let null_literal_expr: Arc<dyn PhysicalExpr> =
-                    Arc::new(Literal::new(ScalarValue::Null));
-
-                let case_expr = CaseExpr::try_new(
-                    None,
-                    vec![(is_null_expr, null_literal_expr)],
-                    Some(array_append_expr),
-                )?;
-                Ok(Arc::new(case_expr))
+                Ok(array_append_expr)
             }
             ExprStruct::ArrayInsert(expr) => {
                 let src_array_expr = self.create_expr(