----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/#review167014 -----------------------------------------------------------
Fix it, then Ship it! Ship It! sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java (line 98) <https://reviews.apache.org/r/56952/#comment239118> Please add comment for these two fields. sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java (line 190) <https://reviews.apache.org/r/56952/#comment239119> Please add comment explaining what's going on here. - Alexander Kolbasov On Feb. 24, 2017, 10:47 p.m., Lei Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56952/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2017, 10:47 p.m.) > > > Review request for sentry. > > > 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 > ----- > > > 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 > >