----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55595/#review162362 -----------------------------------------------------------
Fix it, then Ship it! sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java (line 1) <https://reviews.apache.org/r/55595/#comment233724> I think we should keep the format as "/**" here as the rest header of the files? - Hao Hao On Jan. 17, 2017, 1:43 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55595/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2017, 1:43 a.m.) > > > Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Vamsee > Yarlagadda, and Vadim Spector. > > > Bugs: SENTRY-1428 > https://issues.apache.org/jira/browse/SENTRY-1428 > > > Repository: sentry > > > Description > ------- > > - Added LeaderStatusMonitor to HMSFollower > - Minor cleanup of HMSFollower class > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > f3cefd6a232bfb91db28f04bebcc98ab3c1ca658 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > a89d3f6c52e3e6f23794f3a8998ea8a2236886cb > > Diff: https://reviews.apache.org/r/55595/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
