> On Oct. 13, 2017, 4:39 p.m., Na Li wrote: > > should SentryService log the sentry service is ready? You only logged > > HMSFollower is ready
In order to figure out the service is ready than we need to do it on the HMSFollower because it needs to know when it finishes getting an snapshot. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62979/#review187979 ----------------------------------------------------------- On Oct. 13, 2017, 3:46 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62979/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2017, 3:46 p.m.) > > > Review request for sentry, Alexander Kolbasov and kalyan kumar kalvagadda. > > > Bugs: sentry-1985 > https://issues.apache.org/jira/browse/sentry-1985 > > > Repository: sentry > > > Description > ------- > > Display when Sentry is ready to serve on the console output. > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > 31fd4597dad6b02bb5d8b9f944a488f6f78d7d79 > > > Diff: https://reviews.apache.org/r/62979/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >
