Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10919 )
Change subject: IMPALA-7275: Create table authorization error should not show table name ...................................................................... IMPALA-7275: Create table authorization error should not show table name The authorization correctly checks database level authorization but the exception includes the table name. This fix adds a check for create and modifies the error string to remove the table name. Having an error with CREATE on a table is not correct since to create a table a user needs CREATE on the parent database. Example: SQL: CREATE TABLE db.tbl(col1 INT) Before: User 'user1' does not have privileges to execute 'CREATE' on: db.tbl After: User 'user1' does not have privileges to execute 'CREATE' on: db Testsing: Ran all FE tests Change-Id: I6711a744541abd5ff26574974ba8517b6e51c453 Reviewed-on: http://gerrit.cloudera.org:8080/10919 Reviewed-by: Fredy Wijaya <fwij...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java M fe/src/test/java/org/apache/impala/analysis/AuthorizationStmtTest.java 2 files changed, 50 insertions(+), 40 deletions(-) Approvals: Fredy Wijaya: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10919 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6711a744541abd5ff26574974ba8517b6e51c453 Gerrit-Change-Number: 10919 Gerrit-PatchSet: 6 Gerrit-Owner: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>