[
https://issues.apache.org/jira/browse/IMPALA-14429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18061228#comment-18061228
]
ASF subversion and git services commented on IMPALA-14429:
----------------------------------------------------------
Commit a190eb0a5ed38b17f186a417cc22264161137d73 in impala's branch
refs/heads/master from Steve Carlin
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=a190eb0a5 ]
IMPALA-14429: (addendum) Only int types need extra conversion.
A regression was caused with the IMPALA-14429 commit when a decimal
was being compared with integers in an IN clause. Calcite only
misclassifies integer types, so only those should be changed in
the ImpalaRexBuilder when creating literals.
Change-Id: Ic41e0141737c771b6d5ea48aa99144a919427207
Reviewed-on: http://gerrit.cloudera.org:8080/24031
Reviewed-by: Aman Sinha <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Calcite Planner: 1.40 breaks coercion of smaller numeric datatypes
> ------------------------------------------------------------------
>
> Key: IMPALA-14429
> URL: https://issues.apache.org/jira/browse/IMPALA-14429
> Project: IMPALA
> Issue Type: Sub-task
> Reporter: Steve Carlin
> Priority: Major
> Fix For: Impala 5.0.0
>
>
> In Calcite v1.37, we use coercenodes to change smaller datatypes to match
> Impala types. For instance, the numeric '1' needs to be a tinyint in Impala,
> whereas Calcite makes it an int (CALCITE-7120 is filed for this problem).
> On upgrade from Calcite 1.37 to Calcite 1.40, the mechanism in coercenodes
> fails because of work Calcite did in the AST portion of their code. So we
> need to come up with another way to do this.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
