Hello Aman Sinha, Joe McDonnell, Michael Smith, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/22086

to look at the new patch set (#3).

Change subject: IMPALA-13523: Decimal Precision needs to be in return type
......................................................................

IMPALA-13523: Decimal Precision needs to be in return type

The Inferred return type needs to contain a decimal precision
and scale. In the default case, we grab the precision and scale
off of the first decimal operand.

One query in the e2e tests that will be fixed because of this
is:

select appx_median(c1), appx_median(c2), appx_median(c3)
from decimal_tiny;

Change-Id: Ie10521b587a74930a01c08b711364f897bb2dc33
---
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/operators/CommonOperatorFunctions.java
1 file changed, 16 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/86/22086/3
--
To view, visit http://gerrit.cloudera.org:8080/22086
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie10521b587a74930a01c08b711364f897bb2dc33
Gerrit-Change-Number: 22086
Gerrit-PatchSet: 3
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>

Reply via email to