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

Posted to commits@jackrabbit.apache.org by ju...@apache.org on 2010/09/29 15:14:35 UTC

svn commit: r1002608 - /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/virtual/AbstractVISProvider.java

Author: jukka
Date: Wed Sep 29 13:14:34 2010
New Revision: 1002608

URL: http://svn.apache.org/viewvc?rev=1002608&view=rev
Log:
JCR-2699: Improve read/write concurrency

Revert back to weak references also in AbstractVISProvider

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

Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/virtual/AbstractVISProvider.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/virtual/AbstractVISProvider.java?rev=1002608&r1=1002607&r2=1002608&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/virtual/AbstractVISProvider.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/virtual/AbstractVISProvider.java Wed Sep 29 13:14:34 2010
@@ -71,7 +71,10 @@ public abstract class AbstractVISProvide
      */
     @SuppressWarnings("unchecked")
     private final Map<NodeId, NodeState> nodes =
-        new ReferenceMap(ReferenceMap.HARD, ReferenceMap.SOFT);
+        // Using soft references instead of weak ones seems to have
+        // some unexpected performance consequences, so for now it's
+        // better to stick with weak references.
+        new ReferenceMap(ReferenceMap.HARD, ReferenceMap.WEAK);
 
     /**
      * Listeners (weak references)