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-14 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/#review193839 ---

Re: Review Request 64471: SENTRY-2096: Fail unit tests at end

2017-12-14 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64471/#review193837 --- Ship it! Ship It! - Sergio Pena On Dec. 14, 2017, 7:27