You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ag...@apache.org on 2022/05/03 17:17:55 UTC

[arrow-datafusion] branch master updated: minor: remove redundant code (#2432)

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

agrove 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 c33ffe040 minor: remove redundant code (#2432)
c33ffe040 is described below

commit c33ffe040435cad8f2f76ec78ece1d9092d22211
Author: jakevin <30...@users.noreply.github.com>
AuthorDate: Wed May 4 01:17:50 2022 +0800

    minor: remove redundant code (#2432)
---
 datafusion/core/src/physical_plan/planner.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/datafusion/core/src/physical_plan/planner.rs b/datafusion/core/src/physical_plan/planner.rs
index e3b9bc434..b58029eb5 100644
--- a/datafusion/core/src/physical_plan/planner.rs
+++ b/datafusion/core/src/physical_plan/planner.rs
@@ -798,11 +798,9 @@ impl DefaultPhysicalPlanner {
                     *produce_one_row,
                     SchemaRef::new(schema.as_ref().to_owned().into()),
                 ))),
-                LogicalPlan::SubqueryAlias(SubqueryAlias { input, alias, .. }) => {
+                LogicalPlan::SubqueryAlias(SubqueryAlias { input,.. }) => {
                     match input.as_ref() {
-                        LogicalPlan::TableScan(scan) => {
-                            let mut scan = scan.clone();
-                            scan.table_name = alias.clone();
+                        LogicalPlan::TableScan(..) => {
                             self.create_initial_plan(input, session_state).await
                         }
                         _ => Err(DataFusionError::Plan("SubqueryAlias should only wrap TableScan".to_string()))