[jira] [Resolved] (OAK-264) MicroKernel.diff for depth limited, unspecified changes

["Stefan Guggisberg (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/OAK-264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefan Guggisberg resolved OAK-264.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 0.5

good point! 

fixed in svn r1375476
                
> MicroKernel.diff for depth limited, unspecified changes
> -------------------------------------------------------
>
>                 Key: OAK-264
>                 URL: https://issues.apache.org/jira/browse/OAK-264
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: mk
>            Reporter: Thomas Mueller
>            Assignee: Stefan Guggisberg
>            Priority: Minor
>             Fix For: 0.5
>
>
> Currently the MicroKernel API specifies for the method "diff", if the depth parameter is used, that unspecified changes below a certain path can be returned as:
>   ^ "/some/path"
> I would prefer the slightly more verbose:
>   ^ "/some/path": {}
> Reason: It is similar to how getNode() returns node names if the depth limited: "some":{"path":{}}, and it makes parsing unambiguous: there is always a ':' after the path, whether a property was changed or a node was changed. Without the colon, the parser needs to look ahead to decide whether a node was changed or a property was changed (the token after the path could be the start of the next operation). And we could never ever support ':' as an operation because that would make parsing ambiguous.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira