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 "Angela Schreiber (Jira)" <ji...@apache.org> on 2022/04/06 06:49:00 UTC

[jira] [Updated] (OAK-9666) NodeImpl.hasNode(String relPath) always retrieves the underlying tree

     [ https://issues.apache.org/jira/browse/OAK-9666?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Angela Schreiber updated OAK-9666:
----------------------------------
    Epic Link: OAK-9745

> NodeImpl.hasNode(String relPath) always retrieves the underlying tree
> ---------------------------------------------------------------------
>
>                 Key: OAK-9666
>                 URL: https://issues.apache.org/jira/browse/OAK-9666
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: jcr
>            Reporter: Angela Schreiber
>            Priority: Minor
>
> while discussing JCR-4759 with [~joerghoh], i had a look at _oak-jcr_ and the implementation of {{javax.jcr.Node#hasNode(String relPath)}}, which looks as follows:
> {code}
> @Override
>     public boolean hasNode(String relPath) throws RepositoryException {
>         try {
>             final String oakPath = getOakPathOrThrow(relPath);
>             return perform(new NodeOperation<Boolean>(dlg, "hasNode") {
>                 @NotNull
>                 @Override
>                 public Boolean perform() throws RepositoryException {
>                     return node.getChild(oakPath) != null;
>                 }
>             });
>         } catch (PathNotFoundException e) {
>             return false;
>         }
>     }
> {code}
> this means that the underlaying child tree is always retrieved by calling {{Tree.getChild}} even for those cases where calling {{Tree.hasChild}} would be possible. I don't recall why exactly {{Tree.hasChild}} is never used here but it might we worth taking a look if this would be a performance improvement.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)