You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2016/06/08 13:07:20 UTC
[jira] [Created] (OAK-4444) Reduce number of calls to
NodeBuilder.getNodeState from MergingNodeStateDiff
Michael Dürig created OAK-4444:
----------------------------------
Summary: Reduce number of calls to NodeBuilder.getNodeState from MergingNodeStateDiff
Key: OAK-4444
URL: https://issues.apache.org/jira/browse/OAK-4444
Project: Jackrabbit Oak
Issue Type: Improvement
Components: core
Reporter: Michael Dürig
Assignee: Michael Dürig
Fix For: 1.5.4
The result of the static {{MergingNodeStateDiff.merge}} method is only used in the base case of a recursive diff. In all other cases while traversing the child diffs that result is simply discarded. As calculating the result involves an extra call to {{NodeBuilder.getNodeState}} it inflicts a performance penalty in the case of segment-tar: in that case this call causes the changes in the builder to be written ahead into the store. I figure it is simple enough to specialise the merge method in a way so that call is only done when its result is actually used.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)