From c7be305873b3bcc1db4276960fafdda46794ab51 Mon Sep 17 00:00:00 2001 From: Robert Davies Date: Mon, 10 Apr 2006 17:18:21 +0000 Subject: [PATCH] fix for https://issues.apache.org/activemq/browse/AMQ-687 git-svn-id: https://svn.apache.org/repos/asf/incubator/activemq/trunk@392992 13f79535-47bb-0310-9956-ffa450edef68 --- .../apache/activemq/network/DurableConduitBridge.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/activemq-core/src/main/java/org/apache/activemq/network/DurableConduitBridge.java b/activemq-core/src/main/java/org/apache/activemq/network/DurableConduitBridge.java index 2c984b64e8..40a88b0a7a 100644 --- a/activemq-core/src/main/java/org/apache/activemq/network/DurableConduitBridge.java +++ b/activemq-core/src/main/java/org/apache/activemq/network/DurableConduitBridge.java @@ -54,7 +54,7 @@ public class DurableConduitBridge extends ConduitBridge{ if(isPermissableDestination(dest) && !doesConsumerExist(dest)){ DemandSubscription sub=createDemandSubscription(dest); if(dest.isTopic()){ - sub.getLocalInfo().setSubcriptionName(getLocalBrokerName()+"_"+name); + sub.getLocalInfo().setSubcriptionName(getSubscriberName(dest)); } try{ addSubscription(sub); @@ -81,10 +81,16 @@ public class DurableConduitBridge extends ConduitBridge{ } if(info.isDurable()){ // set the subscriber name to something reproducible - info.setSubcriptionName(getLocalBrokerName()); + + info.setSubcriptionName(getSubscriberName(info.getDestination())); } return doCreateDemandSubscription(info); } + + protected String getSubscriberName(ActiveMQDestination dest){ + String subscriberName = getLocalBrokerName()+"_"+dest.getPhysicalName(); + return subscriberName; + } protected boolean doesConsumerExist(ActiveMQDestination dest){ DestinationFilter filter=DestinationFilter.parseFilter(dest);