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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198528 --- Ship it! Ship It! - Na Li On March 1, 2018, 12:32 p.m., Colm

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198529 --- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentr

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

> 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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65867/#review198544 --- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentr

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

> 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

Sentry Improvment

Hi, This is regarding the sentry improvement for getting groups. As, in sentry, we have the sentry thrift API to get all the roles from a given group, but we don't have thrift API to get all groups from sentry. Can we have these feature in Sentry? Please confirm if I miss anything. -- Regards