Re: Review Request 45859: SENTRY-1120: Show role / privileges info in Sentry Service Webpage

2016-04-12 Thread Lenni Kuff
db/src/main/webapp/SentryService.html (line 65) <https://reviews.apache.org/r/45859/#comment192058> Is it bad the password is cleartext? - Lenni Kuff On April 7, 2016, 3:57 a.m., Li Li wrote: > > --- > This is an auto

Re: Apr Monthly Hangout

2016-04-14 Thread Lenni Kuff
Thanks for the notes Hao On Thu, Apr 14, 2016 at 9:46 PM, Hao Hao wrote: > Hi All, > > We finished Apr monthly hangout. The summary of our discussion today: > >- Disccussed the approach of Sentry-1190. Proposed solution includes a >wrapper around Hive DDLTask. >- Disccussed the statu

Re: Review Request 46666: SENTRY-1217: NPE for list_sentry_privileges_by_authorizable when activeRoleSet is not set

2016-04-25 Thread Lenni Kuff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4/#review130525 --- Ship it! - Lenni Kuff On April 25, 2016, 11:59 p.m., Hao Hao

[ANNOUNCE] New Sentry Committers - Li Li and Colm O hEigeartaigh

2016-05-06 Thread Lenni Kuff
I'm excited to announce that the Apache Sentry PMC has voted to invite Li Li and Colm O hEigeartaigh as committers on the Apache Sentry project. Please join me in congratulating Li and Colm! Thanks, Lenni

Re: Review Request 47569: SENTRY-1265: Sentry service should not require a TGT as it is not talking to other kerberos services as a client

2016-05-20 Thread Lenni Kuff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47569/#review134199 --- Ship it! - Lenni Kuff On May 20, 2016, 12:08 a.m., Sravya

Re: Review Request 47872: SENTRY-1291: SimpleCacheProviderBackend.getPrivileges should return the permission based on authorizationhierarchy

2016-05-25 Thread Lenni Kuff
138) <https://reviews.apache.org/r/47872/#comment199901> authorizationHierarchy This is great - should give a nice perf improvement - Lenni Kuff On May 26, 2016, 5:23 a.m., Hao Hao wrote: > >

Re: Review Request 47872: SENTRY-1291: SimpleCacheProviderBackend.getPrivileges should return the permission based on authorizationhierarchy

2016-05-26 Thread Lenni Kuff
t; authorizationHierarchy - Lenni Kuff On May 26, 2016, 8:19 p.m., Hao Hao wrote: > > --- > This is an automatically generated e-mail. To reply, visi

Re: Review Request 47872: SENTRY-1291: SimpleCacheProviderBackend.getPrivileges should return the permission based on authorizationhierarchy

2016-05-27 Thread Lenni Kuff
> On May 26, 2016, 10:58 p.m., Lenni Kuff wrote: > > sentry-provider/sentry-provider-cache/src/main/java/org/apache/sentry/provider/cache/SimplePrivilegeCache.java, > > line 86 > > <https://reviews.apache.org/r/47872/diff/4/?file=1395744#file1395744line86> > > &

Re: Review Request 47872: SENTRY-1291: SimpleCacheProviderBackend.getPrivileges should return the permission based on authorizationhierarchy

2016-06-01 Thread Lenni Kuff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47872/#review135855 --- Ship it! Ship It! - Lenni Kuff On June 1, 2016, 6:41 a.m

Re: [ANNOUNCE] New Sentry committer - Ashish Singh

2016-08-18 Thread Lenni Kuff
Congrats Ashish and well deserved. Looking forward to your continued contributions. Thanks, Lenni On Thursday, August 18, 2016, Ashish Singh wrote: > Thanks Sravya! > > Happy to be part of this community and looking forward make more > contributions. > > On Thu, Aug 18, 2016 at 10:52 AM, Sravya

Re: How does Sentry authorize SQL Insert

2016-10-13 Thread Lenni Kuff
Hi, That does sound like a potential bug. What privileges have been granted to the user (you can use SHOW CURRENT ROLES and SHOW GRANT ROLE to view the privileges). Can you also share the full INSERT statement? We should follow up with a JIRA if this is an issue. Thanks, Lenni On Thu, Oct 13, 201

Re: Apache Sentry in Maven Central

2016-10-25 Thread Lenni Kuff
Excellent, thanks Colm. On Tue, Oct 25, 2016 at 9:13 AM, Colm O hEigeartaigh wrote: > Hi all, > > I've uploaded the Sentry artifacts for 1.7.0 to the Maven staging repo and > will promote them to Maven Central unless I hear any objections: > > https://repository.apache.org/content/repositories/o

Re: [DISCUSS] Sentry interactive shell

2016-12-14 Thread Lenni Kuff
Thanks for kicking this off Sasha. IMO, a robust Sentry CLI seems super important for the project - especially as Sentry support for more components comes online. It also helps to allow 3rd party integration with Sentry. Is there a design document / JIRA / other material to review on the new CLI?