diff --git a/common/src/main/java/org/apache/rocketmq/common/protocol/route/BrokerData.java b/common/src/main/java/org/apache/rocketmq/common/protocol/route/BrokerData.java index 612e5b411eb921a78976b5ea7a081018d1c1667b..9d868ae03c0da79e9fa4cb9679ec08a8469a064c 100644 --- a/common/src/main/java/org/apache/rocketmq/common/protocol/route/BrokerData.java +++ b/common/src/main/java/org/apache/rocketmq/common/protocol/route/BrokerData.java @@ -15,9 +15,7 @@ * limitations under the License. */ -/** - * $Id: BrokerData.java 1835 2013-05-16 02:00:50Z vintagewang@apache.org $ - */ + package org.apache.rocketmq.common.protocol.route; import java.util.HashMap; @@ -29,6 +27,16 @@ public class BrokerData implements Comparable { private String brokerName; private HashMap brokerAddrs; + public BrokerData() { + + } + + public BrokerData(String cluster, String brokerName, HashMap brokerAddrs) { + this.cluster = cluster; + this.brokerName = brokerName; + this.brokerAddrs = brokerAddrs; + } + public String selectBrokerAddr() { String value = this.brokerAddrs.get(MixAll.MASTER_ID); if (null == value) { diff --git a/namesrv/src/main/java/org/apache/rocketmq/namesrv/routeinfo/RouteInfoManager.java b/namesrv/src/main/java/org/apache/rocketmq/namesrv/routeinfo/RouteInfoManager.java index 16b7847ae922b175a25fe535a270c25a8aac18f6..5a953a9e20e8bcbba1fee7ddf6761f99548786fc 100644 --- a/namesrv/src/main/java/org/apache/rocketmq/namesrv/routeinfo/RouteInfoManager.java +++ b/namesrv/src/main/java/org/apache/rocketmq/namesrv/routeinfo/RouteInfoManager.java @@ -125,11 +125,7 @@ public class RouteInfoManager { BrokerData brokerData = this.brokerAddrTable.get(brokerName); if (null == brokerData) { registerFirst = true; - brokerData = new BrokerData(); - brokerData.setBrokerName(brokerName); - HashMap brokerAddrs = new HashMap(); - brokerData.setBrokerAddrs(brokerAddrs); - + brokerData = new BrokerData(clusterName, brokerName, new HashMap()); this.brokerAddrTable.put(brokerName, brokerData); } String oldAddr = brokerData.getBrokerAddrs().put(brokerId, brokerAddr); @@ -381,9 +377,7 @@ public class RouteInfoManager { for (String brokerName : brokerNameSet) { BrokerData brokerData = this.brokerAddrTable.get(brokerName); if (null != brokerData) { - BrokerData brokerDataClone = new BrokerData(); - brokerDataClone.setBrokerName(brokerData.getBrokerName()); - brokerDataClone.setBrokerAddrs((HashMap) brokerData + BrokerData brokerDataClone = new BrokerData(brokerData.getCluster(), brokerData.getBrokerName(), (HashMap) brokerData .getBrokerAddrs().clone()); brokerDataList.add(brokerDataClone); foundBrokerData = true;