[ 
https://issues.apache.org/jira/browse/IMPALA-14405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18032157#comment-18032157
 ] 

ASF subversion and git services commented on IMPALA-14405:
----------------------------------------------------------

Commit c67b19daf6b47ac380058ea28de71f5d9a7bfe6f in impala's branch 
refs/heads/master from Steve Carlin
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=c67b19daf ]

IMPALA-14405: Labels for Calcite expressions not matching original planner

Calcite sets literal expressions to EXPR$<x> which did not match
expressions given by the Impala planner. For literal expressions
such as "select 1 + 1", Impala creates the column name as "1 + 1".

The field names can be found in the abstract syntax tree, so
they are not set within the CalciteRelNodeConverter before the
logical tree is created.

A small test was added to calcite.test for a basic sanity check,
but more comprehensive tests will be run in the tests/shell module
(e.g. in test_shell_commandline.py and test_shell_interactive) which
contain tests for labels.

Change-Id: Ibd3e6366a284f53807b4b2c42efafa279249c1ea
Reviewed-on: http://gerrit.cloudera.org:8080/23516
Reviewed-by: Steve Carlin <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> Calcite planner: labels for expressions not matching original planner
> ---------------------------------------------------------------------
>
>                 Key: IMPALA-14405
>                 URL: https://issues.apache.org/jira/browse/IMPALA-14405
>             Project: IMPALA
>          Issue Type: Sub-task
>            Reporter: Steve Carlin
>            Assignee: Steve Carlin
>            Priority: Major
>
> The following query:
> select 1 + 1
> was returning $expr0 for the column label.  This needs to match the original 
> planner which has
> 1 + 1
> as the label.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to