Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated March 7, 2018, 10:39 p.m.) Review request for sentry, Alexander Kolbas

Re: Review Request 65945: SENTRY-2164: Convert uses of TransactionBlock to lambdas

2018-03-07 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65945/#review198820 --- Ship it! Ship It! - Steve Moist On March 7, 2018, 7:14 a.m.,

Re: Review Request 65953: SENTRY-2165: NotificationProcesser processAlterTable method has logs wrongly flagged as ERROR when they should be INFO

2018-03-07 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65953/#review198819 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

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

2018-03-07 Thread Steve Moist 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 do

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated March 7, 2018, 7:16 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 65953: SENTRY-2165: NotificationProcesser processAlterTable method has logs wrongly flagged as ERROR when they should be INFO

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65953/#review198797 --- Ship it! Ship It! - Na Li On March 7, 2018, 5:03 p.m., Arjun

Review Request 65953: SENTRY-2165: NotificationProcesser processAlterTable method has logs wrongly flagged as ERROR when they should be INFO

2018-03-07 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65953/ --- Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena. Rep

Re: Review Request 65945: SENTRY-2164: Convert uses of TransactionBlock to lambdas

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65945/#review198793 --- Ship it! Ship It! - Na Li On March 7, 2018, 7:14 a.m., Alexa