Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9942 )
Change subject: IMPALA-6817: Clean up Impala privilege model ...................................................................... IMPALA-6817: Clean up Impala privilege model When we added support for multiple versions of Hadoop, the Impala privilege model was copied from Sentry's Hive privilege model with some modification. This patch aims to clean up that code. Testing: - Added new tests - Ran all front-end tests Cherry-picks: not for 2.x Change-Id: I65f3f9b4d06f3b03bfa2f484737bb746ec598a6b Reviewed-on: http://gerrit.cloudera.org:8080/9942 Reviewed-by: Alex Behm <alex.b...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/compat-minicluster-profile-3/java/org/apache/impala/authorization/ImpalaActionFactory.java M fe/src/compat-minicluster-profile-3/java/org/apache/impala/authorization/ImpalaPrivilegeModel.java M fe/src/main/java/org/apache/impala/authorization/AuthorizationChecker.java M fe/src/main/java/org/apache/impala/authorization/Privilege.java A fe/src/test/java/org/apache/impala/authorization/ImpalaActionFactoryTest.java 5 files changed, 198 insertions(+), 85 deletions(-) Approvals: Alex Behm: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9942 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I65f3f9b4d06f3b03bfa2f484737bb746ec598a6b Gerrit-Change-Number: 9942 Gerrit-PatchSet: 11 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>