Taras Bobrovytsky has submitted this change and it was merged. Change subject: IMPALA-4240: qgen: Add "ParseException line missing )" to Known Errors for Hive ......................................................................
IMPALA-4240: qgen: Add "ParseException line missing )" to Known Errors for Hive There is a bug in Hive with parenthesis parsing for PARTITION BY clauses. Full details can be found in https://issues.apache.org/jira/browse/HIVE-14871. Fixing this issue in Hive isn't trivial, so it should be marked as a "Known Error" in the qgen code. The exception printed by Hive is of the form: FAILED: ParseException line ?:? missing ) at '?' near '?' line ?:? missing EOF at '?' near '?' (state=42000,code=40000) Testing: * All unit tests are passing * Tested against Hive locally * Tested against Impala via the discrepancy searcher Change-Id: Iba3ff4c667ad43ab0927fb19da3dd36ba6504058 Reviewed-on: http://gerrit.cloudera.org:8080/4587 Reviewed-by: Taras Bobrovytsky <[email protected]> Tested-by: Taras Bobrovytsky <[email protected]> --- M tests/comparison/discrepancy_searcher.py 1 file changed, 30 insertions(+), 21 deletions(-) Approvals: Taras Bobrovytsky: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/4587 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba3ff4c667ad43ab0927fb19da3dd36ba6504058 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]>
