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

2017-08-31 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/#review184336 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

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

2017-08-31 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/#review184334 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 61973: SENTRY-1888: Sentry might not fetch all HMS duplicated events IDs when requested

2017-08-31 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61973/#review184333 --- Ship it! Ship It! - Alexander Kolbasov On Aug. 31, 2017, 6:1

Re: Review Request 61973: SENTRY-1888: Sentry might not fetch all HMS duplicated events IDs when requested

2017-08-31 Thread Alexander Kolbasov
> On Aug. 31, 2017, 12:25 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HiveNotificationFetcher.java > > Lines 127 (patched) > > > > > > If I am

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

2017-08-31 Thread Alexander Kolbasov
Sergio, can you share Apache Hive experience with this issue? Thanks, - Alex On Thu, Aug 31, 2017 at 1:10 PM, Sergio Pena wrote: > Hi All, > > This thread discussion is a follow-up to the old thread related to > supporting JDK8 and Datanucleus 4 as the minimum version for Sentry 2.0. > This is

Draft of the quarterly ASF report for Apache Sentry

2017-08-31 Thread Alexander Kolbasov
Here is the draft of the quarterly Apache Sentry ASF report. Please let me know if you have any comments or issues with it. Thanks, - Alex. ## Description: Apache Sentry is a highly modular system for providing fine grained role based authorization to both data and metadata stored on an Apache

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

2017-08-31 Thread Arjun Mishra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/ --- (Updated Aug. 31, 2017, 9:55 p.m.) Review request for sentry, Alexander Kolbaso

Sentry-jdk-1.7 - Build # 826 - Unstable

2017-08-31 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #826) Status: Unstable Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/826/ to view the results.

Re: [DISCUSSION] Move to JDK8 and Datanucleus 4

2017-08-31 Thread Na Li
+1 On Thu, Aug 31, 2017 at 2:51 PM, Sergio Pena wrote: > Thanks Sasha, > > I will create another thread for the JDK8 features discussion then. For > this thread, the JDK8 support as the minimum version for compilation > (without Java8 features) + datanucleus4 is approved then. > > On Wed, Aug 30

[DISCUSS] Allow JDK8 specific features on Sentry 2.0

2017-08-31 Thread Sergio Pena
Hi All, This thread discussion is a follow-up to the old thread related to supporting JDK8 and Datanucleus 4 as the minimum version for Sentry 2.0. This is dedicated to discuss whether we should allow using JDK8 specific features (such as lambda functions and other useful API) in Sentry. Advantag

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

2017-08-31 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62007/#review184312 --- Ship it! As we just discussed, the ideal solution to this probl

Re: [DISCUSSION] Move to JDK8 and Datanucleus 4

2017-08-31 Thread Sergio Pena
Thanks Sasha, I will create another thread for the JDK8 features discussion then. For this thread, the JDK8 support as the minimum version for compilation (without Java8 features) + datanucleus4 is approved then. On Wed, Aug 30, 2017 at 12:42 PM, Alexander Kolbasov wrote: > We are not actively

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

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

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

2017-08-31 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62007/#review184304 --- Maybe I don't understand all the details, but here is what I think

Re: Review Request 61973: SENTRY-1888: Sentry might not fetch all HMS duplicated events IDs when requested

2017-08-31 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61973/ --- (Updated Aug. 31, 2017, 6:13 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 61973: SENTRY-1888: Sentry might not fetch all HMS duplicated events IDs when requested

2017-08-31 Thread Sergio Pena
> On Aug. 31, 2017, 12:25 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Line 247 (original), 251 (patched) > > > > > > Styl

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

2017-08-31 Thread Na Li
> On Aug. 28, 2017, 5:03 p.m., Vamsee Yarlagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Line 135 (original), 126 (patched) > > > > > >

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

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

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

2017-08-31 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62007/ --- (Updated Aug. 31, 2017, 4:14 p.m.) Review request for sentry, Arjun Mishra, Bri