activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tab...@apache.org
Subject svn commit: r1359952 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java
Date Tue, 10 Jul 2012 23:15:06 GMT
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)) {



Mime
View raw message