Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/ --- (Updated Sept. 7, 2017, 1:37 a.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/#review184771 --- Ship it! Ship It! - Alexander Kolbasov On Sept. 7, 2017, 1:2

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/#review184770 --- Ship it! Ship It! - Vamsee Yarlagadda On Sept. 7, 2017, 1:24

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Sergio Pena
> On Sept. 6, 2017, 11:37 p.m., Vamsee Yarlagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HiveNotificationFetcher.java > > Lines 98 (patched) > > > > > > Why can't

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/#review184767 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/ --- (Updated Sept. 7, 2017, 1:24 a.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/#review184757 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

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

Review Request 62136: SENTRY-1928: HMSFollower should close HMS connections when an error to HMS occurs

2017-09-06 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62136/ --- Review request for sentry, Alexander Kolbasov and Vamsee Yarlagadda. Bugs: sent