Steve Carlin has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/24217 )

Change subject: IMPALA-14911: Calcite planner: Fix boolean to numeric comparison
......................................................................

IMPALA-14911: Calcite planner: Fix boolean to numeric comparison

This commit fixes the following failure in the Calcite planner:

select count(*) from functional.alltypestiny
where smallint_col > sleep(100);

The coercion method for binary comparison is overridden to return
the common type as the numeric so that the internals of Calcite
and create the cast around the boolean operand.

Change-Id: I764dcbd2fd0d1323bfb964ed91d94cd3def594c0
Testing: added e2e test
---
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaTypeCoercionImpl.java
M testdata/workloads/functional-query/queries/QueryTest/calcite.test
2 files changed, 21 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/17/24217/2
--
To view, visit http://gerrit.cloudera.org:8080/24217
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I764dcbd2fd0d1323bfb964ed91d94cd3def594c0
Gerrit-Change-Number: 24217
Gerrit-PatchSet: 2
Gerrit-Owner: Steve Carlin <[email protected]>

Reply via email to