You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Sergio Pena via Review Board <no...@reviews.apache.org> on 2018/06/26 18:47:14 UTC

Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/
-----------------------------------------------------------

Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.


Bugs: sentry-2282
    https://issues.apache.org/jira/browse/sentry-2282


Repository: sentry


Description
-------

Removed sentry-binding-hive-v2 and sentry-tests-hive-v2


Diffs
-----

  sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
  sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
  sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
  sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
  sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
  sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
  sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 


Diff: https://reviews.apache.org/r/67747/diff/1/


Testing
-------


Thanks,

Sergio Pena

Re: Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

Posted by Na Li via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/#review205449
-----------------------------------------------------------


Ship it!




Ship It!

- Na Li


On June 26, 2018, 6:47 p.m., Sergio Pena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67747/
> -----------------------------------------------------------
> 
> (Updated June 26, 2018, 6:47 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.
> 
> 
> Bugs: sentry-2282
>     https://issues.apache.org/jira/browse/sentry-2282
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Removed sentry-binding-hive-v2 and sentry-tests-hive-v2
> 
> 
> Diffs
> -----
> 
>   sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
>   sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
>   sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
>   sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
>   sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 
> 
> 
> Diff: https://reviews.apache.org/r/67747/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>

Re: Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

Posted by Sergio Pena via Review Board <no...@reviews.apache.org>.

> On June 27, 2018, 3:18 p.m., Na Li wrote:
> > was the hive-authz copied from hive-authzv2? If not, it is better to save hive-authzv2 in a backup branch just in case, and then remove it.

All the hive-authzv2 is already on hive-authz. The copy on on branch-2 too, so there is no need to keep these modules around anymore.


- Sergio


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/#review205441
-----------------------------------------------------------


On June 26, 2018, 6:47 p.m., Sergio Pena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67747/
> -----------------------------------------------------------
> 
> (Updated June 26, 2018, 6:47 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.
> 
> 
> Bugs: sentry-2282
>     https://issues.apache.org/jira/browse/sentry-2282
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Removed sentry-binding-hive-v2 and sentry-tests-hive-v2
> 
> 
> Diffs
> -----
> 
>   sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
>   sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
>   sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
>   sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
>   sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 
> 
> 
> Diff: https://reviews.apache.org/r/67747/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>

Re: Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

Posted by Na Li via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/#review205441
-----------------------------------------------------------



was the hive-authz copied from hive-authzv2? If not, it is better to save hive-authzv2 in a backup branch just in case, and then remove it.

- Na Li


On June 26, 2018, 6:47 p.m., Sergio Pena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67747/
> -----------------------------------------------------------
> 
> (Updated June 26, 2018, 6:47 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.
> 
> 
> Bugs: sentry-2282
>     https://issues.apache.org/jira/browse/sentry-2282
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Removed sentry-binding-hive-v2 and sentry-tests-hive-v2
> 
> 
> Diffs
> -----
> 
>   sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
>   sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
>   sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
>   sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
>   sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 
> 
> 
> Diff: https://reviews.apache.org/r/67747/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>

Re: Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

Posted by Sergio Pena via Review Board <no...@reviews.apache.org>.

> On June 27, 2018, 12:55 p.m., kalyan kumar kalvagadda wrote:
> > Did you make sure to see of these tests can be moved to sentry-binding-hive?

Yes, I did. There are no new tests, so it is safely to remove.


- Sergio


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/#review205425
-----------------------------------------------------------


On June 26, 2018, 6:47 p.m., Sergio Pena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67747/
> -----------------------------------------------------------
> 
> (Updated June 26, 2018, 6:47 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.
> 
> 
> Bugs: sentry-2282
>     https://issues.apache.org/jira/browse/sentry-2282
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Removed sentry-binding-hive-v2 and sentry-tests-hive-v2
> 
> 
> Diffs
> -----
> 
>   sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
>   sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
>   sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
>   sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
>   sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 
> 
> 
> Diff: https://reviews.apache.org/r/67747/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>

Re: Review Request 67747: SENTRY-2282: Remove hive-authzv2 binding and tests modules completely

Posted by kalyan kumar kalvagadda via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67747/#review205425
-----------------------------------------------------------



Did you make sure to see of these tests can be moved to sentry-binding-hive?

- kalyan kumar kalvagadda


On June 26, 2018, 6:47 p.m., Sergio Pena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67747/
> -----------------------------------------------------------
> 
> (Updated June 26, 2018, 6:47 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.
> 
> 
> Bugs: sentry-2282
>     https://issues.apache.org/jira/browse/sentry-2282
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Removed sentry-binding-hive-v2 and sentry-tests-hive-v2
> 
> 
> Diffs
> -----
> 
>   sentry-binding/pom.xml 17b0f1afa658203327668b33990e5da53dd75ad0 
>   sentry-binding/sentry-binding-hive-v2/pom.xml f77e64ebf86d6095dd56594f37c8b41928918938 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/hadoop/hive/ql/exec/SentryFilterDDLTask.java 0b1acf12d8e9a33226aa00a1d2665ec1beb49593 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java f6b4518da0dbfb62530ef91e0cb24216c9faca29 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookBaseV2.java 5a21dd3ec242d645c391757c9738bc0b31aaedaf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java fac6ba3c650039c2b4f2123cf97ba5eeea4189ab 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java 5a47da8251038cdc1b9a3b9cb52eec4109856da5 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java 61278fe0de778314da50bbb36ccfb152932a3d12 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java 8b56c49abf5e0acac58465267c5cde68c0021a41 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHiveAuthorizationTaskFactoryImplV2.java 2d4bf643663c4ba7f8273436f6c7e10f921e1981 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryHivePrivilegeObject.java 62773855cb55341f3a2f7d80eea2d745a34abb1e 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java f21f9205e14145a874873b1d9befc2a5b5040278 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java 6c2410b063ff0bdebd3c3e6b15a224e6b158bfe1 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java 26fdac803a05715f3ceae363adeea337c1a04abf 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java 7bf7b87223de09a9c0f35fe82d24fa759cb2fa2d 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java 14b952f551d7e2d2d1596a5a01372834b7bed0d4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreBaseV2.java e8e1d233574483af8a4a88df8bf3c4c49fc61bbb 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java 45edf43e0298513a4bb883f83be1a788e9d3ebce 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingBaseV2.java 88fa4a1a6170ec53f2902c46e887b07c935f9edd 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java 107fe1fd425b1bb48030a051403c635d77276a55 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryHiveMetaStoreClientV2.java 1d7a4eb071cde66494528c98e2af9cd89f6bacd7 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetaStoreFilterHook.java 86bf7aac733aa45d27dcdd71bf5b95302b6f6ce8 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerBaseV2.java 642e8733ff4e7fcba41d5e567e13e451e688ba2a 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java ad649c3cb10ef8640fe36c134455b2e6ace6d4a4 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SentryAuthorizerUtil.java 32479d896cdc28ded6eb4d9a7a073482cdcdcf1f 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java 85afe52992b8c95f8aa23219ef89bfb80475c3b3 
>   sentry-binding/sentry-binding-hive-v2/src/test/java/org/apache/sentry/binding/hive/v2/DummyHiveAuthenticationProvider.java 9335c37bd5619405aaeda94a58388be1a596bdba 
>   sentry-tests/sentry-tests-hive-v2/pom.xml b129ed67dd2869317fdee33e18844f376f0fb1a5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java 4c416f724d349e5bd6bca5c2c0aa2a719aeeb0d8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 22d4cf85d11c3e3cf59a6e658ab6c1d456fea982 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java f7f60c768baab2a7cf468880e03fb1bd012f3407 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 4a55b5d6d50294b08b525b1e1f0f0edf77d133f5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java 35870c3bb85f1d7164323f6b5ba5f6556b23611e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbComplexView.java 8ee93ccb04c9710002d4f53c14490f4fdfb85e41 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbConnections.java 867b84c9523e079adea9fab37b2999ee53e54b09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossDbOps.java 0aa166c2c3436e814243080c3e7a5a57d765698b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbDDLAuditLog.java ffc37c4e184d2cebf3617dd35b4fbe82d8831fdd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java a0450af5be3c1463e3a3a27fcfedb0c5c7c52d46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbExportImportPrivileges.java 43064ee34ddaadf2eb8145b8ba2762ee5fb0f7ae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbJDBCInterface.java a26e90a2c9732aca7625e56f9b462118063496be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataObjectRetrieval.java ec99b300792c846255e82a74a7c64fb81dd72666 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMetadataPermissions.java 97b0e06213892caedb097291b523e61d9b2f75f2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbMovingToProduction.java cd27a4f009ba0f942880425624cf58bc965e7359 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbOperations.java 3fab3441cad1678312e3e054a73b0a55525e3e09 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeAtTransform.java 1bb82ed8142f843482cc4b6806648292a6b04a45 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 767bcbe02e0d511f52bc869e7b6a1ee1e6584a5a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtColumnScope.java 659c61feaf5e9fd9ef83ed4a114f3ddeed7fc308 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtDatabaseScope.java 883bedd95982ad92e91d9b5d53297714343f9391 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtFunctionScope.java cebad987c89c7950e28c700c3fe398d409280163 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegesAtTableScope.java a4f07df8df5c26583d99d43c24b1436adf7060d7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbRuntimeMetadataRetrieval.java 8d98179eaba511edf22b073791d11c1ecb108431 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java fa429e75d5148b16450c6b4465a6038c63982e0b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java ee7840bcdc96c07c98833bb3e165b78a7e00e3af 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbUriPermissions.java 1a90e06eab592c529d4f1ed63e3d203277466cd7 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java 5364937d0d5322c40d3c52ca7c997362f5115b4c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithGrantOption.java b25fa2f848d9c4195f5fe2eca5b000d7693824b5 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java a1607d53dc1b3c7efec79a47774f0c497f4f4043 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ad452c6c4d95eff0bc5c2e997eaa2d96dbb5e775 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java 92c0693e6275a0fc0d788c63a2fb9e50eda0586a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 38c1c8b6a186ec47053265c4884bd6cc3ab891cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java 2e508d63782b68f2b7b30c6fe978ca9f3e5b97dd 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java 99614e08c3400790c9fdf68bb7db391b4d11cf6e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java 1c9511d0792f82fc330b7608cdbff10545c6e100 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java 9861a59f2ea16b7fbcfde3fbf561b1c56a4a352a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/StaticUserGroup.java 8306e95311f8a2c04c39fb44baebf4d394f076c0 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestConfigTool.java 9423c2350f2802b54866caf3de1531ba9cc59954 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java b123dcd00828b14b70a463f5444e0a11ba7b01ef 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestCustomSerdePrivileges.java 2851ed6e67f8cb418f1d4f5275912e8b3424780e 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java 23577c20a6eb825c0b495e901cadee098147aa55 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java 5242bb12c4947cc23194e23bcabff1c8dad5d501 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestJDBCInterface.java bc5c08be44ffadd27fc6c3dbe9f9da813cf6a875 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestLockPrivileges.java 0e403d854e6db922286ef840bfdcc9a0b9cd7d42 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java 3c23dc4b9f70d63665374f994ce4e670ccd1a021 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java 24acc799f2106166819d569778acd5b37f39ae46 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java a6edf03a62e5c983d3c47b6d5f822de9b0953166 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java eba46fb3e192b7151331ef5690c9c85b93ff1e08 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java d1a34a85e834a5a507b86230f4966e67604ea664 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java c8712e7721e988873ac752c6a6f31078d9d69282 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java b6be09f42aff19082ae1abc5af0313fa74644500 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 310610ee0669d8bc32a0a869c07b0cd3c2047eae 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtColumnScope.java 9aac78c5e970e0ffbf58481cbafb63c8beca8042 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b28b6f464dbc81fb04d5f78734d56d832fef6b11 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java 40d870f1658105be4b3a0da2d32f75907f780897 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java 4c1cd8e8024226a7fd7ff79dfd3f80ba2ae375c8 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestReloadPrivileges.java 6d4e8d3038b81a886544fc905f991003e5df947c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java efb588ee2bde86253d7f96a44697e8a208acc85c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 017adea2e561d83407baafdb0303068015f3fa5f 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java f1fd15eb194a796522be359e5f366aad28b8087a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java f489b634462ae0ea325356e624a6b2b532b53cd2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java a1b89ae1fb644a5142fef2db7a902a06dd9f0902 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java fd8ec56a501a0d2454020b0ff6d767717451b5cb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestViewPrivileges.java 32da2aea3e673c181954cff53eeb49a27a1aa8be 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java e1881b4bfaa5923fd0dab2096f670b22b944f116 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java 77af4329642807283e6cd3d2e3be837c5d3e0dfb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java dda7dba662f7ba8fb9f705066c4bc680a7358f50 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java 52ba09e5e1006ff44a5073bce412ddee0774b24b 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java 88edc08629191d9eee7f3a57be94f0f1fb20e67a 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java 175e84ceac8f24c1faa905d02daac1739e17f320 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java 3afde62dde1bd8bcd1b5d7409704f892d27760bb 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java 45f0ef28bc2393c16dd385cd8e37c133302b2993 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalMetastoreServer.java bf43798133b280daeff5539e4e1a0b299bf59f16 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java f1e6d75b2ae8e20343ff49bd46ef254ed6320ad4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/SentryPolicyProviderForDb.java 0d1af817c1f8cb306d2ff74159ceaec913be0ca2 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestAuthorizingObjectStore.java 815c38ef65f441dea7ca09c876218ff5fa1390ab 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetaStoreWithPigHCat.java f406fd7fc82dd59dbe7c2fca99bb9bd8585f0ed4 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java a9ff95d62bb744f4db204fff60c24ecb769afa07 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java 1596daadae8bbd827aa90f8c87ebd1eb9b715586 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java 91397061e78a6524410d35643dd3a33be8353ecc 
>   sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java 9381e881c8cdd0e6f6f203c30f88c4fd48a09368 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/core-site-for-sentry-test.xml 01b857681a5b0f9db48402bf72a3dab650a7ede2 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/emp.dat 5922b2016293d785b644ae9ee17e83889e943513 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/hadoop 914d3db17b17c5d3876cf0c680fb138b34d394bd 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/kv1.dat 20fb0dcc6c9fdc8e208f99c9538cdb743910d090 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j.properties d94181675a5ea3c91e1314c914da85f783e652f1 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/log4j2.properties a6ddc2974b8b28f0fc73f2798b1bd0cec04eb2e3 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-provider.ini 014d82783b616946d8246b34ce3b45eb2062985f 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/sentry-site.xml d068011ed8e0a0ee09f2ad901fa0dcbed1966e9a 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImport.ini 15fc5bf3bd22460e7aa258d9428993fbee309196 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportAdmin.ini c778d052d9cf2e049508ae70ec66049de99df84c 
>   sentry-tests/sentry-tests-hive-v2/src/test/resources/testPolicyImportError.ini 4d53f2b58b9d0b75fcbc43e5c93ef19dee487300 
>   sentry-tests/sentry-tests-hive-v2/src/test/scripts/scale-test/create-many-dbs-tables.sh dcdddeb95a896ca8470d0b994f5460531e34d113 
> 
> 
> Diff: https://reviews.apache.org/r/67747/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergio Pena
> 
>