[jboss-svn-commits] JBL Code SVN: r8578 - labs/jbosslabs/trunk/portal-extensions/jmm/jmm-common/src/java/org/jboss/forge/jmm/common/tools

[jboss-svn-commits at lists.jboss.org]

Author: wrzep
Date: 2006-12-22 18:11:49 -0500 (Fri, 22 Dec 2006)
New Revision: 8578

Modified:
   labs/jbosslabs/trunk/portal-extensions/jmm/jmm-common/src/java/org/jboss/forge/jmm/common/tools/Tools.java
Log:
JBLAB-599

Fix for JMM service hot redeployment issues.

Pawel


Modified: labs/jbosslabs/trunk/portal-extensions/jmm/jmm-common/src/java/org/jboss/forge/jmm/common/tools/Tools.java
===================================================================
--- labs/jbosslabs/trunk/portal-extensions/jmm/jmm-common/src/java/org/jboss/forge/jmm/common/tools/Tools.java	2006-12-22 23:10:25 UTC (rev 8577)
+++ labs/jbosslabs/trunk/portal-extensions/jmm/jmm-common/src/java/org/jboss/forge/jmm/common/tools/Tools.java	2006-12-22 23:11:49 UTC (rev 8578)
@@ -95,6 +95,16 @@
 	public static void startJMXService(String jmxServiceURL) {
 		
 		try {
+			//		The MBeanServer to which the JMXConnectorServer will be registered in
+			MBeanServer server = MBeanServerLocator.locate();
+			
+			// 		Make sure the RMI connector isn't already registered
+			ObjectName cntorServerName = ObjectName.getInstance("connectors:protocol=rmi");
+			if (server.isRegistered(cntorServerName)) {
+				log.warn("MBeanRemoteProxy already registred.");
+				return;
+			}
+			
 			//		 The address of the connector
 			JMXServiceURL address = new JMXServiceURL(jmxServiceURL);
 			
@@ -103,11 +113,7 @@
 
 			JMXConnectorServer cntorServer = JMXConnectorServerFactory.newJMXConnectorServer(address, environment, null);
 
-			//		 The MBeanServer to which the JMXConnectorServer will be registered in
-			MBeanServer server = MBeanServerLocator.locate();
-
 			//		 Register the JMXConnectorServer in the MBeanServer
-			ObjectName cntorServerName = ObjectName.getInstance("connectors:protocol=rmi");
 			server.registerMBean(cntorServer, cntorServerName);
 
 			//		 Start the JMXConnectorServer