----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64820/ -----------------------------------------------------------
(Updated Dec. 25, 2017, 10:49 p.m.) Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vadim Spector. Repository: sentry Description ------- Once SentryHMSNotification table is not empty, there are two cases when a full snapshot is triggered. If first event in list of notifications received from HMS greater than latest sentry hms notification Id If latest sentry notification Id is greater than current hms notification Id The later is a unexpected case where for some reason sentry gets ahead of HMS. We should add a logic to trigger a full snapshot in case 2 only after a configurable number of retries. This will avoid unnecessary full snapshot triggers as they are resource intensive Diffs ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java a9afb151a sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java aa1b6a31c sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java edde886a7 Diff: https://reviews.apache.org/r/64820/diff/3/ Testing (updated) ------- mvn -f sentry-provider/sentry-provider-db/pom.xml test Thanks, Arjun Mishra
