Sentry-jdk-1.7 - Build # 818 - Fixed

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

Sentry-jdk-1.7 - Build # 817 - Unstable

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

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

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

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61858/#review183702 --- Ship it! Ship It! - Vamsee Yarlagadda On Aug. 23, 2017, 9:37

Re: Review Request 61862: SENTRY-1822 Allow multiple Sentry reporters.

2017-08-23 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61862/#review183698 --- Ship it! Ship It! - Vamsee Yarlagadda On Aug. 23, 2017, 9:20

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

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

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

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

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/#review183680 --- Ship it! Ship It! - Vadim Spector On Aug. 23, 2017, 10:08 p.

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/ --- (Updated Aug. 23, 2017, 10:08 p.m.) Review request for sentry, Alexander Kolbas

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61858/ --- (Updated Aug. 23, 2017, 9:37 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61858/#review183671 --- Ship it! Ship It! - Alexander Kolbasov On Aug. 23, 2017, 7:4

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Na Li
> On Aug. 23, 2017, 8:22 p.m., Alexander Kolbasov wrote: > > We get all path changes via notifications. DOes this match the way > > notifications are defined in HMS? Yes. I choose the types in Sentry based on HMS types - Na --- This is

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Na Li
> On Aug. 23, 2017, 8:22 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Line 302 (original), 302 (patched) > > > > > >

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

2017-08-23 Thread Arjun Mishra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/ --- Review request for sentry, Alexander Kolbasov, Vamsee Yarlagadda, and Vadim Spec

Review Request 61862: SENTRY-1822 Allow multiple Sentry reporters.

2017-08-23 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61862/ --- Review request for sentry, Brian Towles, Na Li, Sergio Pena, Vamsee Yarlagadda,

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

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

Re: Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

2017-08-23 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61858/#review183653 --- We get all path changes via notifications. DOes this match the way

Review Request 61858: SENTRY-1898: Sentry no longer supports creating more than ~15 partitions at once

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

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/#review183649 --- Ship it! Ship It! - Alexander Kolbasov On Aug. 23, 2017, 7:2

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/#review183647 --- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hd

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/ --- (Updated Aug. 23, 2017, 7:29 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/#review183646 --- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hd

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Alexander Kolbasov
> On Aug. 23, 2017, 6:35 p.m., Alexander Kolbasov wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java > > Lines 537 (patched) > > > > > > Looks like most callers just care whe

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Misha Dmitriev
> On Aug. 23, 2017, 12:50 a.m., Alexander Kolbasov wrote: > > Overall looks good. This are partial comments - I still need more time to > > go through all the changes. These comments are all nits. > > > > Can you add a block comment at the top of HMSPaths.java explaining the > > trickery and t

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Misha Dmitriev
> On Aug. 23, 2017, 6:35 p.m., Alexander Kolbasov wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java > > Lines 537 (patched) > > > > > > Looks like most callers just care whe

Re: Review Request 61827: SENTRY-1892: Reduce memory consumption of HMSPath and TPathEntry

2017-08-23 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61827/#review183637 --- This is the last comment I have - looks good! sentry-hdfs/sentry