[
https://issues.apache.org/jira/browse/SLING-5371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15115464#comment-15115464
]
Bertrand Delacretaz commented on SLING-5371:
--------------------------------------------
I have committed your patch in revision 1726629, thanks!
And made a few minor changes and added some tests in 1726633, 1726635, 1726652
[[email protected]] do you know why the RootFolderMoveListenerTest.testMove()
fails, did I miss something obvious? I've set it to @Ignored for now as it
fails for me.
> Improving JcrInstaller's move listener registration
> ---------------------------------------------------
>
> 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_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)
