Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9911 )
Change subject: IMPALA-6650: Add fine-grained DROP privilege ...................................................................... IMPALA-6650: Add fine-grained DROP privilege Add support for executing DROP statements by granting DROP privilege. These are the new GRANT/REVOKE statements introduced at server, database, and table scopes. GRANT DROP on SERVER svr TO ROLE testrole; GRANT DROP on DATABASE db TO ROLE testrole; GRANT DROP on TABLE tbl TO ROLE testrole; REVOKE DROP on SERVER svr FROM ROLE testrole; REVOKE DROP on DATABASE db FROM ROLE testrole; REVOKE DROP on TABLE tbl FROM ROLE testrole; Testing: - Added new front-end tests - Ran front-end tests Cherry-picks: not for 2.x Change-Id: Iea1dfb0b117cb1725e9656b9a79a9aebee82b5e4 Reviewed-on: http://gerrit.cloudera.org:8080/9911 Reviewed-by: Alex Behm <[email protected]> Tested-by: Impala Public Jenkins --- M common/thrift/CatalogObjects.thrift M fe/.gitignore M fe/src/main/cup/sql-parser.cup M fe/src/main/java/org/apache/impala/analysis/DropFunctionStmt.java M fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java M fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java M fe/src/main/java/org/apache/impala/authorization/Privilege.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeAuthStmtsTest.java M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java M fe/src/test/java/org/apache/impala/analysis/ParserTest.java M fe/src/test/resources/authz-policy.ini.template 11 files changed, 159 insertions(+), 32 deletions(-) Approvals: Alex Behm: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9911 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iea1dfb0b117cb1725e9656b9a79a9aebee82b5e4 Gerrit-Change-Number: 9911 Gerrit-PatchSet: 6 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Adam Holley <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins
