You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2006/08/23 21:34:54 UTC

svn commit: r434135 - /incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java

Author: gnodet
Date: Wed Aug 23 12:34:54 2006
New Revision: 434135

URL: http://svn.apache.org/viewvc?rev=434135&view=rev
Log:
SM-554: Fix deployment problem when one SU in a SA fail.  Other SUs may not be properly shutdown.

Modified:
    incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java

Modified: incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java?rev=434135&r1=434134&r2=434135&view=diff
==============================================================================
--- incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java (original)
+++ incubator/servicemix/trunk/servicemix-common/src/main/java/org/apache/servicemix/common/AbstractDeployer.java Wed Aug 23 12:34:54 2006
@@ -18,6 +18,7 @@
 
 import org.apache.commons.logging.Log;
 
+import javax.jbi.JBIException;
 import javax.jbi.management.DeploymentException;
 
 /**
@@ -48,8 +49,16 @@
         msg.setException(e);
         return new DeploymentException(ManagementSupport.createComponentMessage(msg));
     }
-    
+
     public void undeploy(ServiceUnit su) throws DeploymentException {
+        // Force a shutdown of the SU
+        // There is no initialized state, so after deployment, the SU
+        // is shutdown but may need a cleanup.
+        try {
+            su.shutDown();
+        } catch (JBIException e) {
+            throw new DeploymentException("Unable to shutDown service unit", e);
+        }
     }
-    
+
 }