Re: Review Request 66336: SENTRY-2190: Have verbose debug logs in CounterWait class

2018-03-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66336/ --- (Updated March 30, 2018, 11:10 p.m.) Review request for sentry and Na Li. Cha

Re: Review Request 66376: SENTRY-2193: Synchronize thrift definition with the generated sources

2018-03-30 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66376/#review200251 --- Ship it! Ship It! - Na Li On March 30, 2018, 4:33 p.m., kaly

Re: Review Request 66373: SENTRY-2194: Upgrade Sentry hadoop-version dependency to 2.7.5 to take advantage of security vulnerability fix

2018-03-30 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66373/#review200248 --- Ship it! Ship It! - Na Li On March 30, 2018, 7:05 a.m., Arju

Review Request 66376: SENTRY-2193: Synchronize thrift definition with the generated sources

2018-03-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66376/ --- Review request for sentry, Alexander Kolbasov and Na Li. Bugs: SENTRY-2193

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

2018-03-30 Thread kalyan kumar kalvagadda via Review Board
> On March 14, 2018, 6:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java > > Line 363 (original), 363 (patched) > > > >

Re: Review Request 66360: SENTRY-2192: supress date value in @Generated annotation generated by thrift

2018-03-30 Thread kalyan kumar kalvagadda via Review Board
> On March 29, 2018, 5:29 p.m., Steve Moist wrote: > > Tested this out with some by adding the thrift apis I wrote (new endpoints > > and objects). It generates only the necessary files and changes in most > > cases. When adding new endpoints and objects it shifts things around in > > the a

Review Request 66373: SENTRY-2194: Upgrade Sentry hadoop-version dependency to 2.7.5 to take advantage of security vulnerability fix

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