You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by al...@apache.org on 2022/08/16 00:43:55 UTC

[arrow-datafusion] branch master updated: [Minor] Remove some redundant code (#3169)

This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/master by this push:
     new 8e9a8d553 [Minor] Remove some redundant code (#3169)
8e9a8d553 is described below

commit 8e9a8d553dfb25d4f3e9bf307fce9686522f5721
Author: Andrew Lamb <an...@nerdnetworks.org>
AuthorDate: Mon Aug 15 20:43:52 2022 -0400

    [Minor] Remove some redundant code (#3169)
---
 datafusion/physical-expr/src/planner.rs | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/datafusion/physical-expr/src/planner.rs b/datafusion/physical-expr/src/planner.rs
index 030233e07..44fe0595a 100644
--- a/datafusion/physical-expr/src/planner.rs
+++ b/datafusion/physical-expr/src/planner.rs
@@ -286,19 +286,13 @@ pub fn create_physical_expr(
 
                 let list_exprs = list
                     .iter()
-                    .map(|expr| match expr {
-                        Expr::Literal(ScalarValue::Utf8(None)) => create_physical_expr(
+                    .map(|expr| {
+                        create_physical_expr(
                             expr,
                             input_dfschema,
                             input_schema,
                             execution_props,
-                        ),
-                        _ => create_physical_expr(
-                            expr,
-                            input_dfschema,
-                            input_schema,
-                            execution_props,
-                        ),
+                        )
                     })
                     .collect::<Result<Vec<_>>>()?;