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

Posted to commits@maven.apache.org by mk...@apache.org on 2008/01/13 09:41:13 UTC

svn commit: r611548 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java

Author: mkleint
Date: Sun Jan 13 00:41:12 2008
New Revision: 611548

URL: http://svn.apache.org/viewvc?rev=611548&view=rev
Log:
comment on multithreading.

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java?rev=611548&r1=611547&r2=611548&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenRealmManager.java Sun Jan 13 00:41:12 2008
@@ -48,6 +48,10 @@
         this.logger = logger;
     }
 
+    //mkleint: the clearing is fine for sequenced operations. Even though the 
+    // MavenRealmManager is associated with request, the paralel execution will
+    // eventualy fail as the ClassWorld and PlexusContainer are not meant for 
+    // multithreaded environment.
     public void clear()
     {
         Collection realms = new HashSet( world.getRealms() );