----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60926/ -----------------------------------------------------------
(Updated July 18, 2017, 7:31 p.m.) Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and Na Li. Changes ------- Addressed Lina's comments Bugs: sentry-1760 https://issues.apache.org/jira/browse/sentry-1760 Repository: sentry Description ------- The patch will set the 'requestHmsSnapshot' to TRUE whenever the following cases are found: * List of notifications received are different than expected. This may happen when Sentry has been down or HDFS sync was disabled for a while (more than 24h), and the HMS cleared old notifications (older than 24h) not processed by Sentry causing a gap when retrieving notifications. * Latest Sentry notification ID processed is bigger than current HMS notification ID. This may happen when the HMS DB data was reset or restore from an old snapshot causing sync issues with Sentry. New snapshots are persisted with a new generation ID (or image number), so there's is no need to clean-up older snapshots. Diffs (updated) ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java 083ee4c247f96d5c87b44b9785663a2783e6644d sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryHMSClient.java 05518e81f52965dc1ff102dcdd446010381b9a7a sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java d67c16258c67aae997de4c0451c8b642ab05d298 Diff: https://reviews.apache.org/r/60926/diff/3/ Changes: https://reviews.apache.org/r/60926/diff/2-3/ Testing ------- Added test cases on TestHmsFollower Thanks, Sergio Pena