Re: Sentry Improvment

2018-03-05 Thread Kalyan Kumar Kalvagadda
Sachin, Are you taking about API that returns all the groups which have been granted the role provided? -Kalyan -Kalyan On Sat, Mar 3, 2018 at 1:02 AM, Sachin Dawre wrote: > Hi, > > This is regarding the sentry improvement for getting groups. As, in sentry, > we have the sentry thrift API to

Re: Sentry Improvment

2018-03-05 Thread Stephen Moist
We have an open ticket SENTRY-1558 for the same thing “show groups” from Hive, but it’s blocked because Hive doesn’t support that syntax. This has been requested a few times in the past. Just adding an API call isn’t very user friendly for most other use cases. We can have Hive implement this

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-05 Thread Colm O hEigeartaigh
> On March 2, 2018, 7:34 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAuthorizerFactory.java > > Line 34 (original), 34 (patched) > >

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-05 Thread Colm O hEigeartaigh
> On March 2, 2018, 3:43 p.m., Na Li wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryHiveAuthorizerFactory.java > > Line 34 (original), 34 (patched) > > > > > > c

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-05 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/ --- (Updated March 5, 2018, 5:25 p.m.) Review request for sentry. Bugs: SENTRY-21

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-05 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198647 --- Ship it! Ship It! - kalyan kumar kalvagadda On March 5, 2018

Re: Review Request 65867: SENTRY-2147 - Fix Javadoc for SentryHiveAuthorizerFactory

2018-03-05 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198650 --- Ship it! Ship It! - Steve Moist On March 5, 2018, 5:25 p.m.,

Re: Review Request 65639: SENTRY-1242 Enable getting all privileges on a hive object

2018-03-05 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65639/ --- (Updated March 5, 2018, 9:40 p.m.) Review request for sentry. Repository: sen

Alternative Sentry architecture

2018-03-05 Thread Alexander Kolbasov
I was thinking about possible alternatives to the way Sentry works with HMS. What do people think of Sentry being a proxy to HMS thrift protocol? This way it can see all HMS requests and responses and can enforce HMS-level privileges without any plugins (Hiveserver2 still needs plugins) and it woul