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

angela commented on OAK-7470:
-----------------------------

[~rombert], thanks a lot for your thoughts.... yes... maybe it would be to keep 
it separately... e.g. an interface similar to the {{ReadOnly}} and others in 
the same package. I wouldn't add it to the {{Tree}} interface though because 
it's really just mean to be used for the read-only trees that are just a tree 
wrapper around the {{NodeState}}.

> Remove Usage of ImmutableTree and AbstractTree in Security Code
> ---------------------------------------------------------------
>
>                 Key: OAK-7470
>                 URL: https://issues.apache.org/jira/browse/OAK-7470
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: authorization-cug, core, security-spi
>            Reporter: angela
>            Assignee: angela
>            Priority: Major
>              Labels: m12n
>         Attachments: OAK-7470-tests.patch, OAK-7470.patch
>
>
> With a minor extension to {{TreeProvider}} we would be able to get rid of the 
> direct casting to implementation details like {{ImmutableTree}} and 
> {{AbstractTree}} altogether.
> [~stillalex], patch for review will follow right away.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to