> On July 29, 2016, 8:03 p.m., Sravya Tirukkovalur wrote: > > Seems like we need to not block Sentry startup for the sake of HMS full > > update (to avoid dependencies at startup), but would like to not serve > > requests ( eventually only for Hive requests) until the full update > > processing is finished?
Yeah, put it as todo and filed a upstream for tracking this SENTRY-1437. - Hao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/#review144165 ----------------------------------------------------------- On July 29, 2016, 1:34 a.m., Hao Hao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50182/ > ----------------------------------------------------------- > > (Updated July 29, 2016, 1:34 a.m.) > > > Review request for sentry, Anne Yu and Sravya Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > Added logic in HMSFollower for hive full update. > Block sentry service from start up until full update fetch is complete. > Fetch notification logs which are before failover but did not applied. > > > Diffs > ----- > > sentry-hdfs/sentry-hdfs-common/pom.xml > e767e06c13be2312ccd04e39c518ddef4fbf7744 > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/FullUpdateInitializer.java > PRE-CREATION > > sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestFullUpdateInitializer.java > PRE-CREATION > sentry-provider/sentry-provider-db/pom.xml > b8143ffa3adca9e47e7cb092131d65064d57c86b > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > e93e5b45caa1db61d6c5d69312f66f475dd02ee8 > > Diff: https://reviews.apache.org/r/50182/diff/ > > > Testing > ------- > > > Thanks, > > Hao Hao > >
