You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ra...@apache.org on 2006/04/09 00:48:35 UTC
svn commit: r392624 -
/incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
Author: rajdavies
Date: Sat Apr 8 15:48:33 2006
New Revision: 392624
URL: http://svn.apache.org/viewcvs?rev=392624&view=rev
Log:
fix for http://issues.apache.org/activemq/browse/AMQ-683
Modified:
incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
Modified: incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
URL: http://svn.apache.org/viewcvs/incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java?rev=392624&r1=392623&r2=392624&view=diff
==============================================================================
--- incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java (original)
+++ incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java Sat Apr 8 15:48:33 2006
@@ -186,7 +186,7 @@
}
public void unregisterSubscription(Subscription sub){
- ObjectName name=(ObjectName) subscriptionMap.get(sub);
+ ObjectName name=(ObjectName) subscriptionMap.remove(sub);
if(name!=null){
try{
unregisterSubscription(name);
@@ -331,7 +331,7 @@
try{
c[i]=OpenTypeSupport.convert((Message) messages.get(i));
}catch(Throwable e){
- e.printStackTrace();
+ log.error("failed to browse : " + view,e);
}
}
return c;