Sentry-jdk-1.7-v2 - Build # 134 - Still Failing

2017-09-01 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7-v2 (build #134) Status: Still Failing Check console output at https://builds.apache.org/job/Sentry-jdk-1.7-v2/134/ to view the results.

Re: Review Request 62044: SENTRY-1913: Incorrect constraints on AUTHZ_PATHS_MAPPING.AUTHZ_OBJ_NAME

2017-09-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62044/#review184402 --- Ship it! Ship It! - Alexander Kolbasov On Sept. 1, 2017,

Re: Review Request 61975: SENTRY-1886: Add 1.8.0 -> 2.0.0 upgrade SQL scripts

2017-09-01 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61975/#review184398 --- Ship it! Ship It! - Na Li On Sept. 1, 2017, 7:31 p.m.,

Review Request 62044: SENTRY-1913: Incorrect constraints on AUTHZ_PATHS_MAPPING.AUTHZ_OBJ_NAME

2017-09-01 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62044/ --- Review request for sentry, Alexander Kolbasov, Na Li, and Sergio Pena.

Review Request 62043: SENTRY-1914: Update field size in Sentry to Match Hive definition

2017-09-01 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62043/ --- Review request for sentry, Alexander Kolbasov, Brian Towles, Sergio Pena, and

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-09-01 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/ --- (Updated Sept. 1, 2017, 9:53 p.m.) Review request for sentry, Alexander

Re: Review Request 61975: SENTRY-1886: Add 1.8.0 -> 2.0.0 upgrade SQL scripts

2017-09-01 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61975/#review184394 ---

Re: Review Request 62007: SENTRY-1909 Improvements for memory usage when full path snapshot is sent from Sentry to NN

2017-09-01 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62007/#review184387 ---

Re: Review Request 62007: SENTRY-1909 Improvements for memory usage when full path snapshot is sent from Sentry to NN

2017-09-01 Thread Misha Dmitriev
> On Sept. 1, 2017, 5:01 p.m., Sergio Pena wrote: > > The patch looks good. But just wonder, how does this patch have a memory > > improvement? I see the big change is just to return an ArrayList instead of > > a HashSet in the getPathStrings(). How does this make us better? Hi Sergio.

Re: Review Request 62007: SENTRY-1909 Improvements for memory usage when full path snapshot is sent from Sentry to NN

2017-09-01 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62007/#review184366 --- Ship it! The patch looks good. But just wonder, how does this

Re: [DISCUSS] Allow JDK8 specific features on Sentry 2.0

2017-09-01 Thread Sergio Pena
The Hive community started the discussion to support JDK8 in apr/2016 for Hive 2.x versions. There was a mix of decisions to whether use JDK8 100% or just keep compatibility for JDK8. The final decision was to build and test with JDK8 on Hive 2.0 but keep JDK7 compatibility for one release. The

Re: Review Request 61863: SENTRY-1896 - Optimize retrieving roles for groups

2017-09-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/#review184334 ---