Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59241/#review175472 --- Ship it! Ship It! - Alexander Kolbasov On May 19, 2017, 4:35

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/#review175471 --- Ship it! Ship It! - Alexander Kolbasov On May 18, 2017, 3:08

Re: Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

2017-05-18 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59361/#review175468 --- Ship it! Ship It! - Vamsee Yarlagadda On May 19, 2017, 4:46

Re: Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59361/ --- (Updated May 19, 2017, 4:46 a.m.) Review request for sentry, Brian Towles, Hao

Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

2017-05-18 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59241/ --- (Updated May 19, 2017, 4:35 a.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

2017-05-18 Thread Na Li
> On May 19, 2017, 12:02 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > > Lines 324 (patched) > > > > > > So we no longer ha

Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

2017-05-18 Thread Na Li
> On May 19, 2017, 12:02 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Line 137 (original), 136 (patched) > > > > > > We al

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175460 --- Ship it! Ship It! - Vamsee Yarlagadda On May 19, 2017, 1:18

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/ --- (Updated May 19, 2017, 1:18 a.m.) Review request for sentry, Alexander Kolbasov

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Alexander Kolbasov
Sent! On Thu, May 18, 2017 at 5:25 PM Nishith Nand wrote: > Hi Alex, > > I would like and invite to the slack channel too. > > Thanks, > Nishith > > > > On May 18, 2017, at 5:20 PM, Alexander Kolbasov > wrote: > > > > I’ve set up an experimental Slack account for this - please let me know > if

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Nishith Nand
Hi Alex, I would like and invite to the slack channel too. Thanks, Nishith > On May 18, 2017, at 5:20 PM, Alexander Kolbasov wrote: > > I’ve set up an experimental Slack account for this - please let me know if > you are interested and I’ll send an invite. Already sent one for Rams. > > Let

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Alexander Kolbasov
I’ve set up an experimental Slack account for this - please let me know if you are interested and I’ll send an invite. Already sent one for Rams. Let’s consider this a pilot. - Alex > On May 18, 2017, at 1:08 AM, Rams Venkatesh wrote: > > I like slack, used/using slack for kudu > > -Rams >

Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

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

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175448 --- Ship it! Ship It! - Alexander Kolbasov On May 18, 2017, 10:3

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175444 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread kalyan kumar kalvagadda
> On May 18, 2017, 10:21 p.m., Vamsee Yarlagadda wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/UserGroupInformationInitializer.java > > Lines 43-45 (patched) > > > > > >

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Vamsee Yarlagadda
> On May 18, 2017, 10:21 p.m., Vamsee Yarlagadda wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/UserGroupInformationInitializer.java > > Lines 43-45 (patched) > > > > > >

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/ --- (Updated May 18, 2017, 10:35 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175440 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Sentry HDFS requires CDH Hadoop

2017-05-18 Thread Qianbo Huai
We are currently using Sentry cdh5-1.5.1_5.7.2 and matching CDH Hadoop. When we evaluated newer versions of Hadoop, we noticed that the interface / class (such as AuthorizationProvider and DefaultAuthorizationProvider) required by Sentry HDFS plugin, do not exist outside of CDH Hadoop. Investigati

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175430 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 59241: SENTRY-1705: Do not start HMSFollower if Hive isn't configured

2017-05-18 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59241/ --- (Updated May 18, 2017, 9:04 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/ --- (Updated May 18, 2017, 8:57 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/ --- (Updated May 18, 2017, 8:49 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59167: SENTRY-1580: Provide pooled client connection model with HA

2017-05-18 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/#review175427 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 59167: SENTRY-1580: Provide pooled client connection model with HA

2017-05-18 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/#review175425 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/ --- (Updated May 18, 2017, 7:16 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-18 Thread kalyan kumar kalvagadda
> On May 16, 2017, 6:54 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestConcurrentHMSFollowers.java > > Lines 47 (patched) > > > > > > Does this test

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

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

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/ --- (Updated May 18, 2017, 3:08 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59362: SENTRY-1235 - Some pom changes

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59362/#review175379 --- Ship it! Ship It! - Alexander Kolbasov On May 18, 2017, 9:17

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

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

Review Request 59362: SENTRY-1235 - Some pom changes

2017-05-18 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59362/ --- Review request for sentry. Bugs: SENTRY-1235 https://issues.apache.org/jira

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Rams Venkatesh
I like slack, used/using slack for kudu -Rams > On 18 May 2017, at 8:57 am, Alexander Kolbasov wrote: > > Rams, do you have any preference - IRC vs Slack? > > On Thu, May 18, 2017 at 12:11 AM Rams Venkatesh > wrote: > >> I would be very interested having chat account for sentry this helps >>

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Alexander Kolbasov
Rams, do you have any preference - IRC vs Slack? On Thu, May 18, 2017 at 12:11 AM Rams Venkatesh wrote: > I would be very interested having chat account for sentry this helps > people like me who are new, willing to contribute and interact with others > > -Rams > > > On 18 May 2017, at 12:59 am,

Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

2017-05-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59361/ --- Review request for sentry, Brian Towles, Hao Hao, kalyan kumar kalvagadda, Na Li

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

2017-05-18 Thread Rams Venkatesh
I would be very interested having chat account for sentry this helps people like me who are new, willing to contribute and interact with others -Rams > On 18 May 2017, at 12:59 am, Alexander Kolbasov wrote: > > So far I only saw comments from people working at Cloudera. We do have tools > to