Fredy Wijaya has uploaded a new patch set (#4). ( 
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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/95/13295/4
--
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: newpatchset
Gerrit-Change-Id: Icf08ee9e1a83d7597811b1c8585aee5872d0baaf
Gerrit-Change-Number: 13295
Gerrit-PatchSet: 4
Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Austin Nobis <ano...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to