Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/10882 )
Change subject: IMPALA-7254: Inconsistent decimal behavior for IN/BETWEEN predicate ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/10882/3/fe/src/main/java/org/apache/impala/analysis/Analyzer.java File fe/src/main/java/org/apache/impala/analysis/Analyzer.java: http://gerrit.cloudera.org:8080/#/c/10882/3/fe/src/main/java/org/apache/impala/analysis/Analyzer.java@2286 PS3, Line 2286: Collections.sort(sortedExprs, new Comparator<Expr>() { > The code actually puts all the decimals at the beginning. If we handle non- Ok, I think it's fine the way it is written now. http://gerrit.cloudera.org:8080/#/c/10882/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java File fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java: http://gerrit.cloudera.org:8080/#/c/10882/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeExprsTest.java@2451 PS3, Line 2451: 2 as decimal(38, 37) > I have test cases for that in L2464 and L2491. Yeah, but it would be better to have decimals that are incompatible with each other here also. This test case is not adding anything new the way it is written now. Many of these tests would pass even without this patch. -- To view, visit http://gerrit.cloudera.org:8080/10882 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I44f6ea45f765be7201630541354c72fda0a8a98d Gerrit-Change-Number: 10882 Gerrit-PatchSet: 3 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-Comment-Date: Tue, 10 Jul 2018 03:25:01 +0000 Gerrit-HasComments: Yes
