[arrow-datafusion] branch master updated: avoid unnecessary qualified column parsing in CSE optimzer (#1100)

[houqp]

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

houqp 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 90a4c84  avoid unnecessary qualified column parsing in CSE optimzer 
(#1100)
90a4c84 is described below

commit 90a4c8435152841117c957a74ea4d046815d6c6a
Author: QP Hou <q...@scribd.com>
AuthorDate: Sun Oct 10 22:44:17 2021 -0700

    avoid unnecessary qualified column parsing in CSE optimzer (#1100)
    
    We can produce qualified column directly from qualified field.
    There is no need to serialize to string and parse it again with
    sqlpaser.
---
 datafusion/src/optimizer/common_subexpr_eliminate.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datafusion/src/optimizer/common_subexpr_eliminate.rs 
b/datafusion/src/optimizer/common_subexpr_eliminate.rs
index f8381e8..760067b 100644
--- a/datafusion/src/optimizer/common_subexpr_eliminate.rs
+++ b/datafusion/src/optimizer/common_subexpr_eliminate.rs
@@ -270,7 +270,7 @@ fn build_project_plan(
 
     fields.extend_from_slice(input.schema().fields());
     input.schema().fields().iter().for_each(|field| {
-        project_exprs.push(col(&field.qualified_name()));
+        project_exprs.push(Expr::Column(field.qualified_column()));
     });
 
     let mut schema = DFSchema::new(fields)?;