-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60669/
-----------------------------------------------------------
(Updated July 12, 2017, 6:09 p.m.)
Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li,
Sergio Pena, Vamsee Yarlagadda, and Vadim Spector.
Changes
-------
Addressed comments from sasha and also rebased the patch
Bugs: SENTRY-1762
https://issues.apache.org/jira/browse/SENTRY-1762
Repository: sentry
Description
-------
SENTRY_HMS_NOTIFICATION_ID table will be updated with the ID's for every
notification that sentry processes. This data grows with time.
we need to periodically purge this data as it will effect the performance.
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
350c922
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
322197b
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
193c611
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
395cba6
Diff: https://reviews.apache.org/r/60669/diff/3/
Changes: https://reviews.apache.org/r/60669/diff/2-3/
Testing
-------
Added new tests to test the code change added.
Thanks,
kalyan kumar kalvagadda
