----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/ -----------------------------------------------------------
(Updated Feb. 24, 2017, 2:47 p.m.) Review request for sentry. Changes ------- Update the patch to move SentryStore to `SentryService` Bugs: SENTRY-1637 https://issues.apache.org/jira/browse/SENTRY-1637 Repository: sentry Description ------- * Add configuration keys: SENTRY_STORE_DELTA_REMOVAL_PERIOD_SECONDS to enable the background clean thread. * Use ```ScheduledExecutorSerivce``` to periodically involke ```SentryStore.purgeDeltaChanges()``` Diffs (updated) ----- sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessorFactory.java db55b5aa33cebbef235cceca6ccda48603da2a26 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericPolicyProcessorFactory.java 1cce1fc4b9157c76c68b35a292263c5e96270bd2 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java 7fc3ca8bc2444f6cfcbcbabaebf0d3e18ef3d209 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessorFactory.java 691c1fb81cd50b6bf671576b3bba69aff291c008 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java c91051db70a5f606980fb29f780fbc199945e4f3 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ProcessorFactory.java a3bb6ab19de35d3018f1e9a938936b22d5aecb48 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java e6021f182e26f7b15773d64d84263c6da586d7a9 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java 806d03e81a3660a30c6513efbddd2a1610359fc1 Diff: https://reviews.apache.org/r/56952/diff/ Testing ------- mvn test -Dtest=TestSentryStore Thanks, Lei Xu