[jira] [Commented] (OAK-1120) Enhance observation mechanism to support Apache Sling

["Carsten Ziegeler (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OAK-1120?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13811065#comment-13811065 ] 

Carsten Ziegeler commented on OAK-1120:
---------------------------------------

Alex, we know this - that has been discussed back and forth in the mailing list - and we said to tackle the problem from different angles - this solution here allows to maintain the api in Sling with an improved implementation.
And  - as discussed as well - we don't have any conrete profiling information at all. We first need that otherwise we might optimize in one area and totally forget about other areas and then end up in the same boat

> Enhance observation mechanism to support Apache Sling
> -----------------------------------------------------
>
>                 Key: OAK-1120
>                 URL: https://issues.apache.org/jira/browse/OAK-1120
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core
>            Reporter: Carsten Ziegeler
>         Attachments: OAK-1120.patch
>
>
> The current way the jcr listener work in Sling is not optimal as it reads each and every changed node. So I think it really would be great if we could directly get the information from Oak without the need to additionally process.
> The listener currently:
> 1. Compacts observation events into node added, node changed, and node removed events - it also collects added/changed/removed properties from the events.
> 2. For every added and changed node, it reads the node and the primary node type, the sling:resourceType and sling:resourceSuperType property
> 3. There is a special case, if a node has been added/changed with the name "jcr:content" and the parent node is a file node, then the parent node is reported  as changed/added.
> I've no idea how the current diff mechanism works in Oak, but I would assume that the information for 1. are similar to the result of the diff. I would hope that the properties for 2. are there as well as they need to be diffed against the old value. 3. is more tricky and specific, and I think it would be ok if we would do this in Sling as the ratio of files vs other content is pretty low usually.
> If Oak could provide this information, we could leverage that in Sling and would remove the additional reads - which clearly is an improvement. This would require zero changes in applications based on Sling. If Sling is run on Oak, this implementation is used, otherwis the current JCR based on is used.



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