You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by de...@apache.org on 2012/12/05 10:37:17 UTC

svn commit: r1417331 - /activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java

Author: dejanb
Date: Wed Dec  5 09:37:16 2012
New Revision: 1417331

URL: http://svn.apache.org/viewvc?rev=1417331&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-4181 - remove the browser imidiatelly when finished

Modified:
    activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java

Modified: activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java?rev=1417331&r1=1417330&r2=1417331&view=diff
==============================================================================
--- activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java (original)
+++ activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/broker/region/Queue.java Wed Dec  5 09:37:16 2012
@@ -1530,6 +1530,7 @@ public class Queue extends BaseDestinati
                         // are we done browsing? no new messages paged
                         if (!added) {
                             browser.decrementQueueRef();
+                            browserDispatches.remove(browserDispatch);
                         }
                     } catch (Exception e) {
                         LOG.warn("exception on dispatch to browser: " + browserDispatch.getBrowser(), e);