[
https://issues.apache.org/jira/browse/OAK-5557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15848414#comment-15848414
]
Stefan Egli commented on OAK-5557:
----------------------------------
[~mreutegg], yes. IIUC then it does a rebase before the diff. The rebase
results in the timestamps being 'refreshed' to basically now() - for both from
and to. However, for a branch diff it should look at the latest actual branch
commit's timestamp (afaics)...
> incomplete diffManyChildren during commitHook evaluation in a persisted branch
> ------------------------------------------------------------------------------
>
> Key: OAK-5557
> URL: https://issues.apache.org/jira/browse/OAK-5557
> Project: Jackrabbit Oak
> Issue Type: Bug
> Affects Versions: 1.2.14, 1.2.23
> Environment: 'easily' reproducible on 1.2.14
> 'takes longer to' reproduce on 1.2.23
> Reporter: Stefan Egli
> Assignee: Stefan Egli
> Attachments: DocumentNodeStoreBranchesTest.java
>
>
> In oak 1.2 when doing a diffManyChildren as part of running the CommitHooks
> as part of a branch commit, not all child paths are taken into account in
> some cases.
> The problem seems to be related to the [{{_modified}}
> calculation|https://github.com/apache/jackrabbit-oak/blob/jackrabbit-oak-1.2.14/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java#L2242]
> when doing the query for affected child paths. It seems [that
> {{minValue}}|https://github.com/apache/jackrabbit-oak/blob/jackrabbit-oak-1.2.14/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java#L2242]
> is not correctly calculated when doing {{compareAgainstBaseState}} as part
> of a persisted branch.
> Note that the problem can be reproduced very good on 1.2.14, less good but
> still on the latest 1.2 and not in trunk (ie almost 1.6.0).
> Attaching a test case to reproduce
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
