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,

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

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

[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.

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

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: >

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 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) > > > > > >

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, 6:54 a.m.) Review request for sentry, Arjun Mishra,

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
> On Aug. 31, 2017, 6:28 a.m., Misha Dmitriev wrote: > > In general I wonder whether instead of the old Set<> we should now use > > Collection<> rather than the more explicit List<>. I think using a > > supertype like Collection<> makes sense only if it's used for a method > > parameter that

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/#review184238 --- In general I wonder whether instead of the old Set<> we should