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

angela commented on OAK-709:
----------------------------

exactly... 

in order to push this forward, i would like to proceed as follows:

1. i will provide yet another patch that 
- isolates RootImpls access to rootTree#getNodeState in a separate
  method with a TODO
- includes michael's temporary equals-fix (plus a TODO)
2. commit patch unless someone raises major concerns and leave this issue open 
for further work on the TODOs

i run the build including integration tests with both the TmpPermissionProvider 
and the 
PermissionProviderImpl and they passed. unless we have things sorted out i 
would keep
the tmppermissionprovider in place in order not to risk breaking the build or 
unnecessarily
slow down things for everyone as long as this isn't tested. 
                
> Consider moving permission evaluation to the node state level
> -------------------------------------------------------------
>
>                 Key: OAK-709
>                 URL: https://issues.apache.org/jira/browse/OAK-709
>             Project: Jackrabbit Oak
>          Issue Type: Sub-task
>          Components: core
>            Reporter: angela
>         Attachments: 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> OAK-709_2.patch, OAK-709_3.patch, OAK-709-equals_hack.patch, OAK-709.patch, 
> SecureNodeState.java
>
>


--
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