Review Request 54464: SENTRY-1557: getRolesForGroups(), getRolesForUsers() does too many trips to the the DB

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54464/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, and kalyan kumar kalvaga

Sentry questions

I have some questions about how to integrate my project with Sentry. I noticed that you do not have a user@sentry email address so should I send my questions to the dev@sentry or the issues@sentry list? Thanks, Roberta Marton Apache Trafodion

Re: Review Request 54454: SENTRY-1548 Setting GrantOption to UNSET upsets Sentry

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54454/#review158278 --- sentry-service/sentry-service-common/src/gen/thrift/gen-javabean/

Review Request 54454: SENTRY-1548 Setting GrantOption to UNSET upsets Sentry

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54454/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, Vamsee Yarlagadda, and V

Re: Review Request 54445: SENTRY-1547 - It is possible to create a privilege with all empty fields

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54445/#review158238 --- sentry-service/sentry-service-common/src/gen/thrift/gen-javabean/

Re: Review Request 54445: SENTRY-1547 - It is possible to create a privilege with all empty fields

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54445/ --- (Updated Dec. 6, 2016, 10:32 p.m.) Review request for sentry, Alexander Kolbaso

Review Request 54445: SENTRY-1547 - It is possible to create a privilege with all empty fields

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54445/ --- Review request for sentry, Alexander Kolbasov, Colin Ma, Hao Hao, Vamsee Yarlaga

Re: Review Request 54338: SENTRY-1515: Cleanup exception handling in SentryStore

> On Dec. 5, 2016, 9:25 p.m., Vadim Spector wrote: > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/UpdateForwarder.java, > > line 192 > > > > > > a) why isn't it an error? > > b) LOGGER.

Re: Review Request 53872: SENTRY-1525: Provide script to run Sentry directly from the repo

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53872/#review158155 --- Ship it! Ship It! - kalyan kumar kalvagadda On Dec. 3, 2016,