Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-29 Thread Zachary Amsden via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/ --- (Updated Dec. 29, 2017, 11:30 p.m.) Review request for sentry and Na Li.

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-29 Thread Zachary Amsden via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/ --- (Updated Dec. 29, 2017, 9:14 p.m.) Review request for sentry and Na Li.

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-29 Thread Zachary Amsden via Review Board
> On Dec. 19, 2017, 9:26 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBindingHookBase.java > > Line 837 (original), 837 (patched) > >

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-20 Thread Zachary Amsden via Review Board
> On Dec. 19, 2017, 9:26 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBindingHookBase.java > > Line 837 (original), 837 (patched) > >

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-14 Thread Zachary Amsden via Review Board
> On Dec. 14, 2017, 8:41 p.m., Na Li wrote: > > sentry-provider/sentry-provider-common/src/test/java/org/apache/sentry/provider/common/TestNoAuthorizationProvider.java > > Lines 42 (patched) > > > > > > This is not

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-05 Thread Zachary Amsden via Review Board
> On Dec. 5, 2017, 8:28 p.m., Na Li wrote: > > sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java > > Line 101 (original), 108 (patched) > > > > > >

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-04 Thread Zachary Amsden via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/ --- (Updated Dec. 5, 2017, 12:55 a.m.) Review request for sentry and Na Li.

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-04 Thread Zachary Amsden via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/#review192796 ---

Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-04 Thread Zachary Amsden via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/ --- Review request for sentry and Na Li. Repository: sentry Description ---