You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-issues@jackrabbit.apache.org by "Jukka Zitting (JIRA)" <ji...@apache.org> on 2013/12/02 21:22:35 UTC

[jira] [Resolved] (OAK-1252) Lazy privilege access in AccessControlValidator

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

Jukka Zitting resolved OAK-1252.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 0.13
         Assignee: Jukka Zitting

I tested the patch also with ACE updates and since it doesn't add much overhead there, I went ahead and committed the patch in revision 1547174.

> Lazy privilege access in AccessControlValidator
> -----------------------------------------------
>
>                 Key: OAK-1252
>                 URL: https://issues.apache.org/jira/browse/OAK-1252
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core
>            Reporter: Jukka Zitting
>            Assignee: Jukka Zitting
>            Priority: Minor
>             Fix For: 0.13
>
>         Attachments: 0001-OAK-1252-Lazy-privilege-access-in-AccessControlValid.patch
>
>
> The {{AccessControlValidatorProvider.getPrivileges()}} method takes a few percent of the time spent processing small commits. It's not that much, but privileges are only really needed for the normally quite rare commits that modify ACEs. Thus it would make sense to defer the loading of privileges to only when they're actually needed.



--
This message was sent by Atlassian JIRA
(v6.1#6144)