Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13295 )
Change subject: IMPALA-8530: Return more user friendly error messages on unsupported SQL ...................................................................... IMPALA-8530: Return more user friendly error messages on unsupported SQL This patch updates the error messages to return more friendly error messages for the unsupported SQL statements when running with Sentry or Ranger to improve user experience. This patch tries to differentiate between UnsupportedOperationException, which is an internal exception to indicate a programming error vs UnsupportedFeatureException, which is an exception that will be exposed to the users to indicate that a feature is not supported. Testing: - Added tests for unsupported SQL - Ran FE tests - Ran E2E authorization tests Change-Id: Icf08ee9e1a83d7597811b1c8585aee5872d0baaf Reviewed-on: http://gerrit.cloudera.org:8080/13295 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/authorization/ranger/RangerCatalogdAuthorizationManager.java M fe/src/main/java/org/apache/impala/authorization/ranger/RangerImpaladAuthorizationManager.java M fe/src/main/java/org/apache/impala/authorization/sentry/SentryCatalogdAuthorizationManager.java M fe/src/main/java/org/apache/impala/authorization/sentry/SentryImpaladAuthorizationManager.java M fe/src/main/java/org/apache/impala/service/Frontend.java M tests/authorization/test_ranger.py M tests/authorization/test_sentry.py 7 files changed, 84 insertions(+), 36 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/13295 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icf08ee9e1a83d7597811b1c8585aee5872d0baaf Gerrit-Change-Number: 13295 Gerrit-PatchSet: 6 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Austin Nobis <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
