RE: [discuss] ResourceAuthorizationProvider.hasAccess() performance

2016-03-24 Thread Ma, Junjie
Hi Donghoo, For the first experiment, you cached all privileges to local memory, and do the authorization with the cache. But the cache won't refresh. I think it's a temp cache to resolve the performance problem. For Sentry, we should have an overall solution for the local cache, for example, w

Sentry-jdk-1.7 - Build # 615 - Still Unstable

2016-03-24 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #615) Status: Still Unstable Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/615/ to view the results.

Re: [discuss] ResourceAuthorizationProvider.hasAccess() performance

2016-03-24 Thread nazgul33
Hi Colin, The code snippet was just an experiment, to prove that text to Privilege object conversion is consuming tremendous amount of cpu time. ( I didn't try to profile that though ) Sentry loads rules from db or file and update as needed. I didn't read all the relevant code, but i'm sure at som

Re: Default role for a user?

2016-03-24 Thread Bhooshan Mogal
Hi Hao, Apologies for the delayed response. I think SENTRY-711 is support for adding users to roles. What I was thinking of was more on the lines of granting/revoking privileges to users/groups (and not roles) -- so not RBAC. Thanks, Bhooshan On Tue, Mar 22, 2016 at 3:39 PM, Hao Hao wrote: > H

Review Request 45329: SENTRY-1162: Add shell for Sentry Kafka integration

2016-03-24 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45329/ --- Review request for sentry, Dapeng Sun and Hao Hao. Bugs: SENTRY-1162 https:

Re: Review Request 45262: SENTRY-1137: Update hive dependence to 2.0.0

2016-03-24 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45262/#review125375 --- Ship it! Ship It! - Colin Ma On March 24, 2016, 1:33 a.m., D

Re: Review Request 45198: SENTRY-1103: Authorizable names' case sensitivity must be decided by plugins

2016-03-24 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45198/ --- (Updated March 25, 2016, 1:30 a.m.) Review request for sentry. Repository: se

Sentry-jdk-1.7 - Build # 616 - Fixed

2016-03-24 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #616) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/616/ to view the results.