You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pc...@apache.org on 2008/01/30 20:03:40 UTC

svn commit: r616844 - /openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java

Author: pcl
Date: Wed Jan 30 11:03:37 2008
New Revision: 616844

URL: http://svn.apache.org/viewvc?rev=616844&view=rev
Log:
OPENJPA-437. Reverted back to hard-reference ArrayList instead of a concurrent set, since the new logic only performs local mutation.

Modified:
    openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java

Modified: openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java
URL: http://svn.apache.org/viewvc/openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java?rev=616844&r1=616843&r2=616844&view=diff
==============================================================================
--- openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java (original)
+++ openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java Wed Jan 30 11:03:37 2008
@@ -256,8 +256,7 @@
             if (clss.isEmpty())
                 _pcClassNames = Collections.EMPTY_SET;
             else {
-                Collection c = new ConcurrentReferenceHashSet(
-                    ConcurrentReferenceHashSet.HARD);
+                Collection c = new ArrayList(clss.size());
                 for (Iterator itr = clss.iterator(); itr.hasNext();) {
                     Class cls = (Class) itr.next();
                     c.add(cls.getName());