Re: Issue with SimpleCacheProviderBackend

2017-12-18 Thread Colm O hEigeartaigh
Hi, I've done some further analysis of the problem, and I think it is not directly related to SENTRY-1291. The problem manifests in CommonPrivilege.implies(privilege, model). My (cached) privilege looks like: Server=server1->Db=authz->Table=words->Column=*->action=select The "privilege" I want

[VOTE] - Release Apache Sentry 1.7.1

2017-12-18 Thread Colm O hEigeartaigh
This is a vote to release Apache Sentry 1.7.1. Artifacts: https://dist.apache.org/repos/dist/dev/sentry/1.7.1/ Git tag: https://github.com/apache/sentry/tree/release-1.7.1 Issues fixed: https://issues.apache.org/jira/projects/SENTRY/versions/12342308 +1 from me. Colm. -- Colm O

Re: Issue with SimpleCacheProviderBackend

2017-12-18 Thread Colm O hEigeartaigh
Thanks Kalyan! I was thinking that if the cached privilege part does not appear in the requested "part", and if is "all", then we should skip that part and continue on to the next one. But maybe there is a better solution. Colm. On Mon, Dec 18, 2017 at 4:06 PM, Kalyan Kumar Kalvagadda <

Re: Issue with SimpleCacheProviderBackend

2017-12-18 Thread Kalyan Kumar Kalvagadda
Colm, I will look closer into this today and see If i can help you out. -Kalyan On Mon, Dec 18, 2017 at 4:52 AM, Colm O hEigeartaigh wrote: > Hi, > > I've done some further analysis of the problem, and I think it is not > directly related to SENTRY-1291. The problem

Re: Review Request 64661: SENTRY-1944: Optimize DelegateSentryStore.getGroupsByRoles()

2017-12-18 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64661/ --- (Updated Dec. 18, 2017, 4:36 p.m.) Review request for sentry, Alexander

Re: Review Request 64661: SENTRY-1944: Optimize DelegateSentryStore.getGroupsByRoles()

2017-12-18 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64661/ --- (Updated Dec. 18, 2017, 4:34 p.m.) Review request for sentry, Alexander

Re: Review Request 63775: SENTRY-1543 dropOrRenamePrivilegeForAllRoles() has confusing code

2017-12-18 Thread Steve Moist via Review Board
> On Nov. 15, 2017, 7:11 p.m., Sergio Pena wrote: > > The subject of the review board must match the JIRA subject. You could > > either change the JIRA or the Review subject to match the correct fix. > > > > The getUniqueMSentryPrivilege() is still confusing. Any idea of the > > difference in

Re: Review Request 64661: SENTRY-1944: Optimize DelegateSentryStore.getGroupsByRoles()

2017-12-18 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64661/#review194079 ---

Re: Review Request 64661: SENTRY-1944: Optimize DelegateSentryStore.getGroupsByRoles()

2017-12-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64661/#review194080 ---