You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2011/12/22 17:10:50 UTC

svn commit: r1222310 - /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java

Author: rmannibucau
Date: Thu Dec 22 16:10:49 2011
New Revision: 1222310

URL: http://svn.apache.org/viewvc?rev=1222310&view=rev
Log:
synchronizing some destroy methods

Modified:
    openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java

Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java?rev=1222310&r1=1222309&r2=1222310&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java Thu Dec 22 16:10:49 2011
@@ -979,7 +979,7 @@ public class Assembler extends Assembler
         return deployments;
     }
 
-    public void destroy() {
+    public synchronized void destroy() {
 
         try {
             EjbTimerServiceImpl.shutdown();
@@ -1041,7 +1041,7 @@ public class Assembler extends Assembler
         SystemInstance.reset();
     }
 
-    public void destroyApplication(String filePath) throws UndeployException, NoSuchApplicationException {
+    public synchronized void destroyApplication(String filePath) throws UndeployException, NoSuchApplicationException {
         AppInfo appInfo = deployedApplications.remove(filePath);
         if (appInfo == null) {
             throw new NoSuchApplicationException(filePath);
@@ -1049,7 +1049,7 @@ public class Assembler extends Assembler
         destroyApplication(appInfo);
     }
 
-    public void destroyApplication(AppContext appContext) throws UndeployException {
+    public synchronized void destroyApplication(AppContext appContext) throws UndeployException {
         AppInfo appInfo = deployedApplications.remove(appContext.getId());
         if (appInfo == null) {
             throw new IllegalStateException(String.format("Cannot find AppInfo for app: %s", appContext.getId()));
@@ -1057,7 +1057,7 @@ public class Assembler extends Assembler
         destroyApplication(appInfo);
     }
 
-    public void destroyApplication(AppInfo appInfo) throws UndeployException {
+    public synchronized void destroyApplication(AppInfo appInfo) throws UndeployException {
         deployedApplications.remove(appInfo.path);
         logger.info("destroyApplication.start", appInfo.path);