Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10888 )
Change subject: IMPALA-7260: Fix decimal binary predicates ...................................................................... IMPALA-7260: Fix decimal binary predicates When casting the inputs to a function call, we would try to cast non decimal numbers to a specific decimal type even though this is not necessary. The specific decimal type would be calculated by looking at all the inputs. It was possible for this calculation to fail due to some decimal types being incompatible with each other. We solve the problem by casting the non-decimal numerical inputs to a generic getMinResolutionDecimal(). Testing: - Added some analysis tests. Change-Id: Icc442e84cdff74a376ba25bd9897b0b4df5cf0b4 Reviewed-on: http://gerrit.cloudera.org:8080/10888 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/Expr.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java 2 files changed, 23 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/10888 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icc442e84cdff74a376ba25bd9897b0b4df5cf0b4 Gerrit-Change-Number: 10888 Gerrit-PatchSet: 6 Gerrit-Owner: Taras Bobrovytsky <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
