[
https://issues.apache.org/jira/browse/OAK-6379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alex Deparvu updated OAK-6379:
------------------------------
Attachment: OAK-6379.patch
better patch with tests this time. It seems I missed checking the existing
{{DefaultThreeWayConflictHandlerOursTest}} &
{{DefaultThreeWayConflictHandlerTheirsTest}}.
The {{DELETE_DELETED_NODE}} is also interesting as it would not fail before,
but it would set the child node to an empty property and I think it's better to
just remove it.
[~mduerig] for your review!
> NPE in MergingNodeStateDiff on DELETE_DELETED_PROPERTY
> ------------------------------------------------------
>
> Key: OAK-6379
> URL: https://issues.apache.org/jira/browse/OAK-6379
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: core
> Reporter: Alex Deparvu
> Assignee: Alex Deparvu
> Fix For: 1.7.3
>
> Attachments: OAK-6379.patch
>
>
> Relevant trace:
> {noformat}
> java.lang.NullPointerException: null
> at
> com.google.common.base.Preconditions.checkNotNull(Preconditions.java:192)
> ~[guava-15.0.jar:na]
> at
> org.apache.jackrabbit.oak.plugins.memory.MemoryNodeBuilder.setProperty(MemoryNodeBuilder.java:507)
> ~[classes/:na]
> at
> org.apache.jackrabbit.oak.plugins.commit.MergingNodeStateDiff.applyPropertyResolution(MergingNodeStateDiff.java:171)
> ~[classes/:na]
> at
> org.apache.jackrabbit.oak.plugins.commit.MergingNodeStateDiff.resolveConflict(MergingNodeStateDiff.java:126)
> ~[classes/:na]
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
