Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12963
Change subject: IMPALA-8385: Refactor Sentry admin user check ...................................................................... IMPALA-8385: Refactor Sentry admin user check This patch moves the hardcoded Sentry admin user check from the generic "show roles" and "show grant" catalog operations to Sentry authorization plugin, i.e. SentryImpaladAuthorizationManager. This patch also removes isAdmin() from AuthorizationManager interface. Testing: - Added a new authorization E2E test to test for Sentry admin check - Ran all FE tests - Ran all authorization E2E tests Change-Id: I911228b09af7eed5d5dc002b20591ef64dc625d3 --- M be/src/catalog/catalog-server.cc M be/src/catalog/catalog.cc M be/src/catalog/catalog.h M be/src/exec/catalog-op-executor.cc M be/src/exec/catalog-op-executor.h M be/src/service/client-request-state.cc M be/src/service/fe-support.cc M common/thrift/CatalogService.thrift M fe/src/main/java/org/apache/impala/authorization/AuthorizationManager.java M fe/src/main/java/org/apache/impala/authorization/NoneAuthorizationFactory.java M fe/src/main/java/org/apache/impala/authorization/ranger/RangerCatalogdAuthorizationManager.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/FeSupport.java M fe/src/main/java/org/apache/impala/service/JniCatalog.java A tests/authorization/test_sentry.py 16 files changed, 186 insertions(+), 76 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/63/12963/4 -- To view, visit http://gerrit.cloudera.org:8080/12963 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I911228b09af7eed5d5dc002b20591ef64dc625d3 Gerrit-Change-Number: 12963 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <[email protected]>
