-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60669/
-----------------------------------------------------------
(Updated July 8, 2017, 3:31 p.m.)
Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li,
Sergio Pena, Vamsee Yarlagadda, and Vadim Spector.
Changes
-------
Addressed review comments.
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
1402ab1
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
ec938da
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
83f00ca
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
ac266fe
Diff: https://reviews.apache.org/r/60669/diff/2/
Changes: https://reviews.apache.org/r/60669/diff/1-2/
Testing
-------
Added new tests to test the code change added.
Thanks,
kalyan kumar kalvagadda
