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)) {
