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

2016-05-07 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review132161 --- sentry-provider/sentry-provider-common/src/main/java/org/apache/s

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

2016-05-07 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/ --- (Updated May 8, 2016, 1:15 a.m.) Review request for sentry, Dapeng Sun, Gregory

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

2016-05-07 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/ --- (Updated May 8, 2016, 1:02 a.m.) Review request for sentry, Dapeng Sun, Gregory

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

2016-05-07 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/ --- (Updated May 8, 2016, 12:57 a.m.) Review request for sentry, Dapeng Sun, Gregor

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

2016-05-07 Thread Anne Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45550/#review132099 --- Fix it, then Ship it! sentry-binding/sentry-binding-hive-commo

Re: [DISCUSS] Wiki updates on "How to integrate with Sentry"

2016-05-07 Thread Anne Yu
Hi Bhooshan, https://cwiki.apache.org/confluence/display/SENTRY/Integrating+with+Sentry+New+Universal+Authorization+Model Thanks, Anne On Wed, Mar 9, 2016 at 12:35 PM, Bhooshan Mogal wrote: > Happy to review when the doc is available too. May have some feedback from > the work we're currently

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

2016-05-07 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review132143 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

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

2016-05-07 Thread Ashish Singh
> On May 6, 2016, 10 p.m., Gregory Chanan wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/SentryGenericProviderBackend.java, > > line 95 > > > > > > hmm...the SimpleF