[jira] [Updated] (OAK-2912) Clear the modified and deleted map in PermissionHook after processing is complete

["Chetan Mehrotra (JIRA)" <ji...@apache.org>]

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

Chetan Mehrotra updated OAK-2912:
---------------------------------
    Attachment: OAK-2912.patch

[patch|^OAK-2912.patch] which just clears the map when processCommit has applied the changes.

[~anchela] Can you review the patch

> Clear the modified and deleted map in PermissionHook after processing is complete
> ---------------------------------------------------------------------------------
>
>                 Key: OAK-2912
>                 URL: https://issues.apache.org/jira/browse/OAK-2912
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: security
>            Reporter: Chetan Mehrotra
>            Priority: Minor
>             Fix For: 1.3.0
>
>         Attachments: OAK-2912.patch
>
>
> {{PermissionHook}} has in memory state in {{modified}} and {{deleted}} maps. In case of repository migration which is implemented as a large commit this can consume quite a bit of memory. In one of the migration it was taking ~1 GB of memory.
> In a commit involving multiple commit hooks once {{PermissionHook}} has done the work it can clear that state so that memory is not held up untill all the hooks are applied. Specially as IndexingHook takes long time and also has some memory requirements



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)