----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/ -----------------------------------------------------------
(Updated May 11, 2017, 12:55 p.m.) Review request for sentry, Alexander Kolbasov, Hao Hao, Na Li, Sergio Pena, Vamsee Yarlagadda, and Vadim Spector. Changes ------- Addressed review comments. Bugs: SENTRY-1669 https://issues.apache.org/jira/browse/SENTRY-1669 Repository: sentry Description ------- Sentry should store Id's of all the notifications that it processed in a seperate database. HMSFollower before every run should get the last notification ID from this persistent store. Diffs (updated) ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryHmsNotification.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo 8fd5278 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java f590a52 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java ef67865 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java e271ea4 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java e7443eb Diff: https://reviews.apache.org/r/58975/diff/7/ Changes: https://reviews.apache.org/r/58975/diff/6-7/ Testing ------- Sentry stores Id's of all the notifications that it processed in a seperate database. HMSFollower before every run gets the last notification ID from this persistent store. Thanks, kalyan kumar kalvagadda