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

Michael Dürig commented on OAK-315:
-----------------------------------

+1 for separating.

Moving the registration of built in node type to oak-jcr is also fine with me. 
However we need to take care to do that while at the same time keeping 
registration as lazy as it is. Note that access to the latest root creates a 
branch in the Microkernel. In the current implementation this does not happen 
on login itself (*) but is deferred until the session is actually accessed. 
This is also the point where the default node types are registered if not 
already present. 

(*) the reason for this is the infamous repository pinger of Apache Sling. See 
also OAK-272. 
                
> Separate built-in node types from ReadWriteNodeTypeManager
> ----------------------------------------------------------
>
>                 Key: OAK-315
>                 URL: https://issues.apache.org/jira/browse/OAK-315
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core, jcr
>            Reporter: Marcel Reutegger
>            Priority: Minor
>
> Registration code for built-in node types are currently in 
> ReadWriteNodeTypeManager. I think this should be separate from the more 
> generic node type management. Triggering the registration in ContentSession 
> also looks strange to me.
> I suggest we move the built-in node type registration to oak-jcr for now. I 
> think oak-core or more specifically ContentSession shouldn't have to know 
> about built-in node types or plugins in general.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to