Re: Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-05 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59793/#review176974 --- Ship it! Ship It! - Vamsee Yarlagadda On June 4, 2017, 6:48

Re: Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-05 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59793/#review176952 --- Ship it! Ship It! - Na Li On June 4, 2017, 6:48 p.m., Alexan

Re: Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-05 Thread Alexander Kolbasov
> On June 5, 2017, 3:11 a.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 390 (patched) > > > > > > what is the benefit of catching ex

Re: Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-04 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59793/#review176892 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-04 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59793/ --- (Updated June 4, 2017, 6:48 p.m.) Review request for sentry, kalyan kumar kalva

Review Request 59793: SENTRY-1780: FullUpdateInitializer does not kill the threads whenever getFullHMSSnapshot throws an...

2017-06-04 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59793/ --- Review request for sentry, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vams