You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2012/07/11 01:15:06 UTC
svn commit: r1359952 -
/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java
Author: tabish
Date: Tue Jul 10 23:15:05 2012
New Revision: 1359952
URL: http://svn.apache.org/viewvc?rev=1359952&view=rev
Log:
additional fix for fo:r https://issues.apache.org/jira/browse/AMQ-3921 make sure on selector change that we remove the subscription from the list so that the updated one gets put back in.
Modified:
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java
Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java?rev=1359952&r1=1359951&r2=1359952&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java Tue Jul 10 23:15:05 2012
@@ -220,6 +220,9 @@ public class Topic extends BaseDestinati
// Need to delete the subscription
topicStore.deleteSubscription(clientId, subscriptionName);
info = null;
+ synchronized (consumers) {
+ consumers.remove(subscription);
+ }
} else {
synchronized (consumers) {
if (!consumers.contains(subscription)) {