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 "Alex Parvulescu (JIRA)" <ji...@apache.org> on 2013/09/20 17:05:51 UTC
[jira] [Commented] (OAK-1030) Simplify SessionDelegate#getItem
[ https://issues.apache.org/jira/browse/OAK-1030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13773055#comment-13773055 ]
Alex Parvulescu commented on OAK-1030:
--------------------------------------
more tweaks on the patch: we could also replace _name.isEmpty()_ with _denotesRoot(path)_ but that's just cosmetics
> Simplify SessionDelegate#getItem
> --------------------------------
>
> Key: OAK-1030
> URL: https://issues.apache.org/jira/browse/OAK-1030
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: jcr
> Reporter: Alex Parvulescu
> Attachments: SessionDelegate.patch
>
>
> The _getItem_ call generates extra existence verification calls for both property and nodes, which also happen in the delegate #create calls.
> The simplification I'm proposing also has some perf gains:
> {code}
> -- before:
> # ReadDeepTreeTest min 10% 50% 90% max N
> Oak-Tar 216 224 229 237 297 261
> # ConcurrentReadDeepTreeTest min 10% 50% 90% max N
> Oak-Tar 2416 2604 2833 3916 5671 20
> -- after:
> # ReadDeepTreeTest min 10% 50% 90% max N
> Oak-Tar 195 199 204 212 280 292
> # ConcurrentReadDeepTreeTest min 10% 50% 90% max N
> Oak-Tar 1909 1997 2244 2498 2616 27
> {code}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira