You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ke...@apache.org on 2010/01/05 03:16:13 UTC
svn commit: r895854 -
/geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
Author: kevan
Date: Tue Jan 5 02:16:06 2010
New Revision: 895854
URL: http://svn.apache.org/viewvc?rev=895854&view=rev
Log:
GERONIMO-5013 avoid multiple threads from simultaneously adding a resource names to the resourcesNotFound Set. This will avoid potential ConcurrentModificationExceptions
Modified:
geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
Modified: geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java?rev=895854&r1=895853&r2=895854&view=diff
==============================================================================
--- geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java (original)
+++ geronimo/server/branches/2.2/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/config/MultiParentClassLoader.java Tue Jan 5 02:16:06 2010
@@ -567,7 +567,9 @@
// Resource not found -- no need to search for it again
//
if (!resourcesNotFound.contains(name)) {
- resourcesNotFound.add(name);
+ synchronized(resourcesNotFound) {
+ resourcesNotFound.add(name);
+ }
}
return null;