Re: Review Request 62953: SENTRY-1983: Several commit/rollback errors happen in oracle12c-r1 due to current isolation level

2017-10-12 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62953/#review187896 --- Ship it! Ship It! - Alexander Kolbasov On Oct. 12, 2017,

Review Request 62953: SENTRY-1983: Several commit/rollback errors happen in oracle12c-r1 due to current isolation level

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62953/ --- Review request for sentry and Alexander Kolbasov. Bugs: sentry-1983

Review Request 62951: SENTRY-1231: Sentry doesn't secure index location uri, when do "CREATE INDEX LOCATION ''/uri"

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62951/ --- Review request for sentry and kalyan kumar kalvagadda. Bugs: sentry-1231

Re: Review Request 62950: SENTRY-1981: Bump codehale metrics version to latest 3.0.2 version

2017-10-12 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62950/#review187864 --- Ship it! Ship It! - kalyan kumar kalvagadda On Oct. 12,

Re: Review Request 62950: SENTRY-1981: Bump codehale metrics version to latest 3.0.2 version

2017-10-12 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62950/#review187863 --- Ship it! Ship It! - Alexander Kolbasov On Oct. 12, 2017,

Review Request 62950: SENTRY-1981: Bump codehale metrics version to latest 3.0.2 version

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62950/ --- Review request for sentry, Alexander Kolbasov and kalyan kumar kalvagadda.

Re: Review Request 62812: SENTRY-1975 - Add sqoop support to SentryShellGeneric

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62812/#review187851 --- Ship it! The patchs looks good. +1 - Sergio Pena On Oct. 6,

Re: Review Request 62902: SENTRY-1978: Move the hive-authz2 grant/revoke implementation into the sentry-binding-hive module

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62902/ --- (Updated Oct. 12, 2017, 7:35 p.m.) Review request for sentry, Alexander

Re: Review Request 62942: Bump the minimum java version to 8

2017-10-12 Thread Sergio Pena via Review Board
> On Oct. 12, 2017, 6:56 p.m., Alexander Kolbasov wrote: > > You also need to update version for com.google.errorprone I left a comment on the jira about it. I couldn't bump the google_error_prone dependency to any one that uses Java8 because it was causing IncompatibleClassChangeError when

Re: Review Request 62942: Bump the minimum java version to 8

2017-10-12 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62942/#review187835 --- You also need to update version for com.google.errorprone -

Review Request 62942: Bump the minimum java version to 8

2017-10-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62942/ --- Review request for sentry, Alexander Kolbasov, Colm O hEigeartaigh, and kalyan