----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62902/ -----------------------------------------------------------
Review request for sentry, Alexander Kolbasov, Colm O hEigeartaigh, and kalyan kumar kalvagadda. Bugs: sentry-1978 https://issues.apache.org/jira/browse/sentry-1978 Repository: sentry Description ------- This patch is moving some hive-authz2 profile classes related to the grant/revoke tasks to allow running the current tests with it and start doing the switch to authz2. The patch does the following: - Stop using the SentryGrantRevokeTask for grant/revoke task execution. - Use SentryHiveAccessController to execute similar tasks that SentryGrantRevokeTask used to have. - Configure the new controller on the HiveAuthzBindingSessionHook class. - Configure tests to run the authz2 access controller. Diffs ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java 3454910db1950f11e3317011bf4c08041a4ec5ac sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingSessionHook.java 994ae7a852d36653eb642112da7c0c58952f2f33 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java ceb3b17714d5dfc4c6186b5f9cf536d6ddbb662b sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHivePrivilegeObject.java PRE-CREATION sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java PRE-CREATION sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAccessController.java PRE-CREATION sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAuthorizerFactory.java f6297e9a19e4624cfc9c5a57d939e5873261263d sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAuthorizerImpl.java 9c72876abbde2d1217503b90dfbfcd6d609427a8 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/util/SentryAuthorizerUtil.java PRE-CREATION sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java a62a0a66f1894f9039f099691b9fcfa2e98d8549 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java 27cfba9fab49f44f74f7b7d24564b22e3ac437ba sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 35cb2bb4ffb9109721ba24e6dac84667bfdefa37 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java 5cd69e17b3d70dfc7b739354e9fe21a5f7678120 Diff: https://reviews.apache.org/r/62902/diff/1/ Testing ------- All tests passed. Thanks, Sergio Pena