Review Request 47084: SENTRY-1233: Logging improvements to SentryConfigToolSolr

2016-05-06 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47084/ --- Review request for sentry and Vamsee Yarlagadda. Repository: sentry

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericProviderBackend.

2016-05-06 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review132087 ---

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

2016-05-06 Thread Sravya Tirukkovalur
Well deserved! Congratulations Colm and Li! And thanks for your contributions so far! Looking forward to your continued contributions! On Fri, May 6, 2016 at 10:25 AM, Lenni Kuff wrote: > I'm excited to announce that the Apache Sentry PMC has voted to invite Li > Li and

Review Request 47075: SENTRY-1208: Make HOST implied in privileges if not specified explicitly.

2016-05-06 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47075/ --- Review request for sentry, Colin Ma and Dapeng Sun. Bugs: SENTRY-1208

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

2016-05-06 Thread Ashish Singh
Congrats Li and Colm! On Fri, May 6, 2016 at 10:37 AM, Chao Sun wrote: > Congrats Li and Colm! > > On Fri, May 6, 2016 at 10:25 AM, Lenni Kuff wrote: > > > I'm excited to announce that the Apache Sentry PMC has voted to invite Li > > Li and Colm O

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericProviderBackend.

2016-05-06 Thread Ashish Singh
> On May 6, 2016, 8:39 a.m., Dapeng Sun wrote: > > sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/generic/service/thrift/TSentryPrivilege.java, > > line 62 > > > > > >

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericProviderBackend.

2016-05-06 Thread Ashish Singh
> On May 6, 2016, 8:34 a.m., Colin Ma wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/SentryGenericProviderBackend.java, > > line 123 > > > > > > For the filter,

Re: Review Request 45550: Fix some "major" issues identified by Sonarqube

2016-05-06 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45550/ --- (Updated May 6, 2016, 10:28 a.m.) Review request for sentry. Repository:

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericProviderBackend.

2016-05-06 Thread Dapeng Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review131996 ---

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericProviderBackend.

2016-05-06 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review132002 ---

Re: Review Request 47012: SENTRY-1222: Improve SentryIniPolicyFileFormatter to support user section in .ini file

2016-05-06 Thread Dapeng Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47012/#review131995 --- Ship it! Ship It! - Dapeng Sun On 五月 5, 2016, 9:59 p.m.,