Re: Review Request 56271: SENTRY-1621: HMSFollower to retry connecting to HMS upon connection loss

2017-02-03 Thread Vamsee Yarlagadda
> On Feb. 3, 2017, 8:28 p.m., Hao Hao wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java, > > line 265 > > > > > > Could there be any other exception impile

Re: Review Request 56271: SENTRY-1621: HMSFollower to retry connecting to HMS upon connection loss

2017-02-03 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56271/#review164158 --- Fix it, then Ship it! sentry-provider/sentry-provider-db/src/m

Re: Review Request 56271: SENTRY-1621: HMSFollower to retry connecting to HMS upon connection loss

2017-02-02 Thread Vamsee Yarlagadda
> On Feb. 3, 2017, 5:03 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java, > > line 269 > > > > > > can client.close() throw any e

Re: Review Request 56271: SENTRY-1621: HMSFollower to retry connecting to HMS upon connection loss

2017-02-02 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56271/#review164087 --- Fix it, then Ship it! Ship It! sentry-provider/sentry-provide

Review Request 56271: SENTRY-1621: HMSFollower to retry connecting to HMS upon connection loss

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