jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "angela (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (JCR-2672) Cache also failed principal lookups
Date Thu, 15 Jul 2010 10:18:52 GMT

     [ https://issues.apache.org/jira/browse/JCR-2672?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

angela resolved JCR-2672.
-------------------------

    Resolution: Fixed

Revision: 964362
Applied a modified version of the patch provided by jukka:
- no enabled in AbstractPrincipalProvider by default
- added config option
- DefaultPrincipalProvider enables negative-cache unless it is explicitly turn off/on by the
config.
- in addition to the patch the negative-entry cached required some adjustment to the cache-clearing
mechanism in
  the DefaultPrincipalProvider.
- finally also added a couple of tests.

> Cache also failed principal lookups
> -----------------------------------
>
>                 Key: JCR-2672
>                 URL: https://issues.apache.org/jira/browse/JCR-2672
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core, security
>    Affects Versions: 2.0.0, 2.1.0
>            Reporter: Jukka Zitting
>            Assignee: angela
>            Priority: Minor
>             Fix For: 2.2.0
>
>         Attachments: JCR-2672.patch
>
>
> The principal cache in Jackrabbit normally does a good job in ensuring good performance
in critical areas like ACL evaulation. However, the cache only includes successful principal
lookups, so an ACE that references a missing (or mistyped) principal can cause notable performance
issues as a new principal lookup is needed whenever the node covered by such an ACL is accessed.
> To solve that problem I propose that we extend the principal cache to also cover negative
principal lookups.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message