[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting/transport ...
Ron Sigal
ron_sigal at yahoo.com
Fri May 18 22:33:19 EDT 2007
User: rsigal
Date: 07/05/18 22:33:19
Modified: src/main/org/jboss/remoting/transport Tag: remoting_2_x
Connector.java
Log:
JBREM-747: init() will always pass MBeanServer reference to ServerInvoker, and stop() will unregister ServerInvoker MBean.
Revision Changes Path
No revision
No revision
1.27.2.2 +15 -2 JBossRemoting/src/main/org/jboss/remoting/transport/Connector.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: Connector.java
===================================================================
RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/transport/Connector.java,v
retrieving revision 1.27.2.1
retrieving revision 1.27.2.2
diff -u -b -r1.27.2.1 -r1.27.2.2
--- Connector.java 20 Jan 2007 06:35:16 -0000 1.27.2.1
+++ Connector.java 19 May 2007 02:33:19 -0000 1.27.2.2
@@ -111,7 +111,7 @@
* @author <a href="mailto:d_jencks at users.sourceforge.net">David Jencks</a>
* @author <a href="mailto:juha at jboss.org">Juha Lindfors</a>
* @author <a href="mailto:tom at jboss.org">Tom Elrod</a>
- * @version $Revision: 1.27.2.1 $
+ * @version $Revision: 1.27.2.2 $
* @jmx.mbean description = "An MBean wrapper around a ServerInvoker."
* @jboss.xmbean
*/
@@ -404,8 +404,12 @@
if (!server.isRegistered(objName))
{
server.registerMBean(invoker, objName);
- invoker.setMBeanServer(server);
}
+ else
+ {
+ log.warn(objName + " is already registered with MBeanServer");
+ }
+ invoker.setMBeanServer(server);
}
catch (Throwable e)
{
@@ -737,6 +741,15 @@
{
if (invoker != null)
{
+ try
+ {
+ ObjectName objName = new ObjectName(invoker.getMBeanObjectName());
+ server.unregisterMBean(objName);
+ }
+ catch (Exception e)
+ {
+ log.error("invalid Object Name", e);
+ }
invoker.stop();
invoker.destroy();
InvokerRegistry.destroyServerInvoker(invoker);
More information about the jboss-cvs-commits
mailing list