You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ho...@apache.org on 2005/12/08 09:46:36 UTC
svn commit: r355040 -
/geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java
Author: hogstrom
Date: Thu Dec 8 00:46:34 2005
New Revision: 355040
URL: http://svn.apache.org/viewcvs?rev=355040&view=rev
Log:
Caught ugly Exception at shutdown and silently accepted it as it is expected
Modified:
geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java
Modified: geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java?rev=355040&r1=355039&r2=355040&view=diff
==============================================================================
--- geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java (original)
+++ geronimo/trunk/modules/jmx-remoting/src/java/org/apache/geronimo/jmxremoting/JMXConnector.java Thu Dec 8 00:46:34 2005
@@ -124,9 +124,18 @@
}
public void doStop() throws Exception {
- server.stop();
- server = null;
- log.debug("Stopped JMXConnector " + url);
+ try {
+ server.stop();
+ } catch (java.io.IOException e) {
+ // java.io.IOException is expected.
+ } catch (Exception e) {
+ // Otherwise, something bad happened. Rethrow the exception.
+ throw e;
+ }
+ finally {
+ server = null;
+ log.debug("Stopped JMXConnector " + url);
+ }
}
public void doFail() {