-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46909/
-----------------------------------------------------------
(Updated May 12, 2016, 8:06 a.m.)
Review request for sentry, Dapeng Sun, Gregory Chanan, Hao Hao, and Sravya
Tirukkovalur.
Changes
-------
Remove cache interface and use reflection to create privilege converter
Bugs: SENTRY-1229
https://issues.apache.org/jira/browse/SENTRY-1229
Repository: sentry
Description
-------
SENTRY-1229: Add caching to SentryGenericProviderBackend.
Diffs (updated)
-----
pom.xml 334f98bf1dc96a4db5e82ca04d443b617b138c72
sentry-binding/sentry-binding-hive-common/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHookBase.java
df68dd3baa2774e4d74f43e2677b4ecfa5179502
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java
57fc1e4914cd83f17cd65d9fbbaeedac94b6d837
sentry-binding/sentry-binding-kafka/src/main/java/org/apache/sentry/kafka/conf/KafkaAuthConf.java
2f4f8dff330ae647f70f383a557d6ac43eef3364
sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/CacheProvider.java
PRE-CREATION
sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ProviderBackend.java
ffd3af4903dd59f37ea1ff4a55138742fdfa74da
sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/TableCache.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryMappingData.java
9b70abdfc40576989a4038648f438b1d9a74b40c
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/SentryGenericProviderBackend.java
222b6fd530a0e26ea625d1be0fd68b0828558316
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/UpdatableCache.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java
d51b3baf58de85633755598b55d78d6488fc5041
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericPolicyProcessor.java
2a287e9b95e36b7d365de603d94208bad59966bc
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClient.java
60502895a0fcdd781f5bd61b29a676a7c96f81b8
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java
dce3dade7f42fe35a849612c5caf2e98d2dac578
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
f680bc7c2855a6e1f82518de0061244f156e3d98
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClientDefaultImpl.java
0da3cfd9276c48993a24423f9b3511278888776f
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java
a922d3bad36fc497abc9ae3e8f007e79ea358b2e
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
00e3fbde76ebf84704ee110adfca30869845a7b8
sentry-provider/sentry-provider-db/src/main/resources/sentry_policy_service.thrift
2088eae176c0704effdabbbd91a90d693f4da15b
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/thrift/TestSentryGenericServiceIntegration.java
fcf0e7b9dc45eb12cc1b76f3084efd6340c039a4
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStoreImportExport.java
8dadc2abc24db027911601b566aa8ec8d728d1e8
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceImportExport.java
865f66e44eeaed9cec81333c93df180146cb370d
sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/SimpleFileProviderBackend.java
91fd4a3b37c145982bc395077df8778e941f99ed
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsBase.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsExtMaxGroups.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsMaxGroups.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
d2a1d36ee0572d5d24c200f631607e4cfa8168e3
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java
8818c4c2b35b5a9979b7ca11a8a8548121bc348a
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart1.java
b474484710f969bc1fc44793d4a60760f11708e4
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java
e1881b4bfaa5923fd0dab2096f670b22b944f116
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/MiniDFS.java
77af4329642807283e6cd3d2e3be837c5d3e0dfb
sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaTestServer.java
2a3b317b03b1cfa9394b503277ee4f81488dcc55
sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/AbstractKafkaSentryTestBase.java
a2cfa28da2af548a872ec0d2e5620ae1c27041b3
sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java
250522ed33394a667b1a0d6c47c68b210e0214a3
Diff: https://reviews.apache.org/r/46909/diff/
Testing
-------
Extended Kafka e2e tests. Will be running perf tests for Kafka-Sentry
integration with caching turned on.
Thanks,
Ashish Singh
