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

(Updated May 12, 2016, 8:31 a.m.)


Review request for sentry, Dapeng Sun, Gregory Chanan, Hao Hao, and Sravya 
Tirukkovalur.


Changes
-------

Fix wrong patch uploaded in last version.


Bugs: SENTRY-1229
    https://issues.apache.org/jira/browse/SENTRY-1229


Repository: sentry


Description
-------

SENTRY-1229: Add caching to SentryGenericProviderBackend.


Diffs (updated)
-----

  
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/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/thrift/SentryGenericServiceClient.java
 76ff15b91fddad6b5aa429d2439eb2a5086544d7 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java
 74b6963ab4015521462fb3c25559d0d12292424f 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
 00e3fbde76ebf84704ee110adfca30869845a7b8 
  
sentry-provider/sentry-provider-file/src/main/java/org/apache/sentry/provider/file/SimpleFileProviderBackend.java
 3c0e349443e3917812b46aa4547c3cab14dab052 
  
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

Reply via email to