Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15332 )
Change subject: IMPALA-7686: Allow RANGE() clause before HASH() clause for PARTITION BY ...................................................................... IMPALA-7686: Allow RANGE() clause before HASH() clause for PARTITION BY Modified the sql_parser.cup to accept the reversed syntax. Testing: -Added extra analyzer tests to cover the case when RANGE() is before HASH() for Kudu tables. Change-Id: I914e340e9acfb0f49c7d4f78705dbd9bde0aec8c Reviewed-on: http://gerrit.cloudera.org:8080/15332 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/cup/sql-parser.cup M fe/src/test/java/org/apache/impala/analysis/AnalyzeKuduDDLTest.java M fe/src/test/java/org/apache/impala/analysis/ParserTest.java 3 files changed, 19 insertions(+), 4 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/15332 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I914e340e9acfb0f49c7d4f78705dbd9bde0aec8c Gerrit-Change-Number: 15332 Gerrit-PatchSet: 9 Gerrit-Owner: Adam Tamas <[email protected]> Gerrit-Reviewer: Adam Tamas <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]>
