-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56720/
-----------------------------------------------------------
(Updated Feb. 17, 2017, 5:26 p.m.)
Review request for sentry, Alexander Kolbasov and Hao Hao.
Changes
-------
This patch removed the thread that periodicially call
```SentryStore#purgeDeltaChangeTables()```. As discussed offline, it should be
moved into `o.a.s.serivce.thrift`. I will add this after we reach the concense
how to organzie `SentryStore` in that package.
Bugs: SENTRY-1611
https://issues.apache.org/jira/browse/SENTRY-1611
Repository: sentry
Description
-------
Provide a background service to periodically purge path and permission delta
change tables.
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryChange.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPathChange.java
952b11f4e81426ec51c0ad6131ad579ba5e3383e
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java
3662bfdebd1a4e2172ab83bd8c8e60ccf21b55b9
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
4cc123fd0de05a055b4276d213eafc0eb070213d
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
e10ddfd0372c985e5a8bd6d5b3ca07d5cb618a68
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
dfaac1555fe729efab3f8fdccb914978001fee56
Diff: https://reviews.apache.org/r/56720/diff/
Testing
-------
Run `mvn test -Dtest=TestSentryStore`
Thanks,
Lei Xu
