[jira] [Resolved] (SLING-5371) JcrInstaller should not listen on / for moves

["Bertrand Delacretaz (JIRA)" <ji...@apache.org>]

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

Bertrand Delacretaz resolved SLING-5371.
----------------------------------------
    Resolution: Fixed

Good catch, the test obviously works for me as well now. I have applied your patch in revision 1728047, marking this ticket fixed, thanks!

> JcrInstaller should not listen on / for moves
> ---------------------------------------------
>
>                 Key: SLING-5371
>                 URL: https://issues.apache.org/jira/browse/SLING-5371
>             Project: Sling
>          Issue Type: Improvement
>          Components: Installer
>            Reporter: Marc Pfaff
>            Assignee: Bertrand Delacretaz
>         Attachments: SLING_5371.patch, SLING_5371_fixing_testclass.patch, SLING_5371_jackrabbit.patch, SLING_5371_jcr_only.patch
>
>
> The JcrInstaller is currently registering a JCR listener for move events on the root node with setting the 'deep' property to true. If it comes to processing the underlaying Oak observation queue, this combination is very inefficient, as it does not allow Oak to filter changes by path. 
> AFAIU this move listener is covering the use case where an installer artifact is moved from/to an installer root folder. I therefore wanted to propose to register the move listener on the configured root folders only.  



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