Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21037 )
Change subject: IMPALA-12811: Exception during re-analyze can be lost ...................................................................... IMPALA-12811: Exception during re-analyze can be lost When there is an AnalysisException during re-analyze, we try to print the re-written and original statements via invoking toSql() on them. But toSql() fails because the analysis of the statement was incomplete, so it throws another exception (typically an IllegalStateException without any relevant information about the original issue). This patch puts the original LOG.error() in a try-catch, then wraps the original AnalysisException into a new exception that just mentions that the error occurred after query rewrite. Testing: * added a column masking test with an invalid masking function Change-Id: Ie6e36b08703c07a2a8d68a4ec0e8ddd65ba03199 Reviewed-on: http://gerrit.cloudera.org:8080/21037 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java M testdata/workloads/functional-query/queries/QueryTest/ranger_column_masking.test M tests/authorization/test_ranger.py 3 files changed, 32 insertions(+), 4 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21037 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie6e36b08703c07a2a8d68a4ec0e8ddd65ba03199 Gerrit-Change-Number: 21037 Gerrit-PatchSet: 4 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
