----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/ -----------------------------------------------------------
(Updated May 12, 2016, 9:03 a.m.) Review request for sentry, Dapeng Sun, Gregory Chanan, Hao Hao, and Sravya Tirukkovalur. Changes ------- Minor cleanup. 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