----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63993/#review191615 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java Lines 218 (patched) <https://reviews.apache.org/r/63993/#comment269458> Please update the log. "Force a full snapshot" is not accurate here as there are no snpshots taken so far. It may be something like "Need to request a full HMS snapshot" - kalyan kumar kalvagadda On Nov. 21, 2017, 5:21 p.m., Arjun Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63993/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2017, 5:21 p.m.) > > > Review request for sentry, kalyan kumar kalvagadda, Na Li, Sergio Pena, and > Vadim Spector. > > > Repository: sentry > > > Description > ------- > > AUTHZ_PATHS_SNAPSHOT_ID is only populated when a full HMS snapshot is > persisted. When initially HDFS Sync is disabled, we will create a full > snapshot but never persist it, and at the same time populate > SENTRY_HMS_NOTIFICATION_ID table. Later when HDFS sync is enabled, a full > snapshot will not occur unless until HMS and Sentry are out of sync. This > will result in ACL's not being applied unless until HMS and Sentry are out of > sync, since we only send NN snapshots if AUTHZ_PATHS_SNAPSHOT_ID has values > greater than 0 > We should create a full snapshot if hdfsSync is enabled, and > AUTHZ_PATHS_SNAPSHOT_ID is empty > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > 4dc2bf6d1 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > c4cc91806 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java > bbcf09382 > > > Diff: https://reviews.apache.org/r/63993/diff/1/ > > > Testing > ------- > > mvn -f sentry-provider/sentry-provider-db/pom.xml test > > > Thanks, > > Arjun Mishra > >