OAK-11969 do not check the existence of the tree twice (#2560)

joerghoh · reschke · commit 3c063ba36e01 · 2026-02-15T20:21:47.000+01:00
diff --git a/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java b/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/delegate/NodeDelegate.java
@@ -339,8 +339,7 @@ public NodeDelegate getChild(String relPath) throws RepositoryException {
     @NotNull
     public Iterator<NodeDelegate> getChildren() throws InvalidItemStateException {
         Iterator<Tree> iterator = getTree().getChildren().iterator();
-        return IteratorUtils.transform(
-                IteratorUtils.filter(iterator, tree -> tree.exists()),
+        return IteratorUtils.transform(iterator, 
                 tree -> new NodeDelegate(sessionDelegate, tree));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -339,8 +339,7 @@ public NodeDelegate getChild(String relPath) throws RepositoryException {`
`339`	`339`	`@NotNull`
`340`	`340`	`public Iterator<NodeDelegate> getChildren() throws InvalidItemStateException {`
`341`	`341`	`Iterator<Tree> iterator = getTree().getChildren().iterator();`
`342`		`- return IteratorUtils.transform(`
`343`		`- IteratorUtils.filter(iterator, tree -> tree.exists()),`
	`342`	`+ return IteratorUtils.transform(iterator,`
`344`	`343`	`tree -> new NodeDelegate(sessionDelegate, tree));`
`345`	`344`	`}`
`346`	`345`