[
https://issues.apache.org/jira/browse/OAK-3615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela resolved OAK-3615.
-------------------------
Resolution: Fixed
Fix Version/s: 1.3.10
Committed revision 1713893 (set to 128 for now)
> Add initial capacity to StringBuilder used to calculate path of a tree
> ----------------------------------------------------------------------
>
> Key: OAK-3615
> URL: https://issues.apache.org/jira/browse/OAK-3615
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: core
> Reporter: angela
> Assignee: angela
> Priority: Minor
> Fix For: 1.3.10
>
>
> while benchmarking permission evaluation on the oak level (without having
> oak-jcr overhead), i spotted that the {{StringBuilder}} used to calculate the
> path of a tree often needs to increase it's capacity. Extract from the
> profiler info reported in oak-run:
> {code}
> at java.util.Arrays.copyOf(Arrays.java:2882)
> at
> java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuilder.java:100)
> at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:390)
> at java.lang.StringBuilder.append(StringBuilder.java:119)
> at
> org.apache.jackrabbit.oak.plugins.tree.impl.AbstractTree.buildPath(AbstractTree.java:206)
> at
> org.apache.jackrabbit.oak.plugins.tree.impl.AbstractTree.buildPath(AbstractTree.java:205)
> at
> org.apache.jackrabbit.oak.plugins.tree.impl.AbstractTree.getPath(AbstractTree.java:197)
> {code}
> Would it make sense to initialize the {{StringBuilder}} with some initial
> capacity? Afaik the default is 16 which doesn't seem sufficient for a default
> path as we are experiencing them in an Oak instance.
> [~mduerig], what do you think?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
