You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by th...@apache.org on 2018/11/21 15:00:28 UTC

svn commit: r1847115 - /jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java

Author: thomasm
Date: Wed Nov 21 15:00:27 2018
New Revision: 1847115

URL: http://svn.apache.org/viewvc?rev=1847115&view=rev
Log:
OAK-7910 Composite node store: fix performance issue

Modified:
    jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java

Modified: jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java?rev=1847115&r1=1847114&r2=1847115&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java (original)
+++ jackrabbit/oak/trunk/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeBuilder.java Wed Nov 21 15:00:27 2018
@@ -223,7 +223,9 @@ class CompositeNodeBuilder implements No
 
     @Override
     public NodeBuilder setChildNode(final String name, NodeState nodeState) {
-        checkState(exists(), "This builder does not exist: " + PathUtils.getName(getPath()));
+        if (!exists()) {
+            throw new IllegalStateException("This builder does not exist: " + PathUtils.getName(getPath()));
+        }
         String childPath = simpleConcat(getPath(), name);
         final MountedNodeStore childStore = ctx.getOwningStore(childPath);
         if (childStore != ctx.getGlobalStore() && !nodeBuilders.get(childStore).exists()) {