Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-01 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/ --- Review request for sentry. Bugs: SENTRY-2147

Re: Review Request 65866: SENTRY-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65866/ --- (Updated March 1, 2018, 11:47 a.m.) Review request for sentry, Alexander

Review Request 65866: SENTRy-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65866/ --- Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar

Re: Review Request 65866: SENTRY-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65866/ --- (Updated March 1, 2018, 11:41 a.m.) Review request for sentry, Alexander

Re: Review Request 65866: SENTRY-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65866/ --- (Updated March 1, 2018, 6:19 p.m.) Review request for sentry, Alexander

Re: Review Request 65866: SENTRY-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65866/#review198460 ---

Re: Review Request 65866: SENTRY-2146: Add better error handling to ResourceAuthorizationProvider and improve logging in related classes

2018-03-01 Thread Arjun Mishra via Review Board
> On March 1, 2018, 5:39 p.m., Steve Moist wrote: > > sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java > > Lines 140 (patched) > > > > > > I