You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by de...@apache.org on 2013/06/28 12:58:31 UTC

svn commit: r1497733 - /activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java

Author: dejanb
Date: Fri Jun 28 10:58:31 2013
New Revision: 1497733

URL: http://svn.apache.org/r1497733
Log:
https://issues.apache.org/jira/browse/AMQ-4609 - register network connectors after mbean server is properly started

Modified:
    activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java

Modified: activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java?rev=1497733&r1=1497732&r2=1497733&view=diff
==============================================================================
--- activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java (original)
+++ activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java Fri Jun 28 10:58:31 2013
@@ -418,9 +418,6 @@ public class BrokerService implements Se
             }
         });
         networkConnectors.add(connector);
-        if (isUseJmx()) {
-            registerNetworkConnectorMBean(connector);
-        }
         return connector;
     }
 
@@ -560,6 +557,9 @@ public class BrokerService implements Se
                 MDC.remove("activemq.broker");
                 try {
                     startManagementContext();
+                    for (NetworkConnector connector : getNetworkConnectors()) {
+                        registerNetworkConnectorMBean(connector);
+                    }
                 } finally {
                     MDC.put("activemq.broker", brokerName);
                 }