jackrabbit-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ste...@apache.org
Subject svn commit: r504623 - /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java
Date Wed, 07 Feb 2007 17:21:02 GMT
Author: stefan
Date: Wed Feb  7 09:21:01 2007
New Revision: 504623

URL: http://svn.apache.org/viewvc?view=rev&rev=504623
Log:
minor performance tweaking

Modified:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java

Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java?view=diff&rev=504623&r1=504622&r2=504623
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java
(original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/SessionImpl.java
Wed Feb  7 09:21:01 2007
@@ -432,12 +432,20 @@
      */
     public Node getNodeByUUID(UUID uuid) throws ItemNotFoundException, RepositoryException
{
         NodeImpl node = getNodeById(new NodeId(uuid));
+        // since the uuid of a node is only exposed through jcr:uuid declared
+        // by mix:referenceable it's rather unlikely that a client can possibly
+        // know the internal uuid of a non-referenceable node; omitting the
+        // check for mix:referenceable seems therefore to be a reasonable
+        // compromise in order to improve performance.
+/*
         if (node.isNodeType(QName.MIX_REFERENCEABLE)) {
             return node;
         } else {
             // there is a node with that uuid but the node does not expose it
             throw new ItemNotFoundException(uuid.toString());
         }
+*/
+        return node;
     }
 
     /**



Mime
View raw message