Re: Review Request 55595: SENTRY-1428: Only leader should follow HMS updates

2017-01-20 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55595/#review162474 --- Ship it! Ship It! - Hao Hao On Jan. 20, 2017, 1:28 a.m.,

Re: Review Request 55595: SENTRY-1428: Only leader should follow HMS updates

2017-01-19 Thread Alexander Kolbasov
> On Jan. 19, 2017, 11:57 p.m., Hao Hao wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java, > > line 1 > > > > > > I think we should keep the format as "/**"

Re: Review Request 55595: SENTRY-1428: Only leader should follow HMS updates

2017-01-19 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55595/ --- (Updated Jan. 20, 2017, 1:28 a.m.) Review request for sentry, Hao Hao, kalyan

Review Request 55595: SENTRY-1428: Only leader should follow HMS updates

2017-01-16 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55595/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Vamsee Yarlagadda,