svn commit: r439298 - /incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java

[ra...@apache.org]

Author: rajdavies
Date: Fri Sep  1 05:49:16 2006
New Revision: 439298

URL: http://svn.apache.org/viewvc?rev=439298&view=rev
Log:
use logger for exceptions

Modified:
    incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java

Modified: incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java
URL: http://svn.apache.org/viewvc/incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java?rev=439298&r1=439297&r2=439298&view=diff
==============================================================================
--- incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java (original)
+++ incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java Fri Sep  1 05:49:16 2006
@@ -36,6 +36,7 @@
 import javax.management.openmbean.TabularType;
 
 import org.apache.activemq.ActiveMQConnectionFactory;
+import org.apache.activemq.broker.BrokerService;
 import org.apache.activemq.broker.jmx.OpenTypeSupport.OpenTypeFactory;
 import org.apache.activemq.broker.region.Destination;
 import org.apache.activemq.command.ActiveMQDestination;
@@ -45,8 +46,11 @@
 import org.apache.activemq.filter.BooleanExpression;
 import org.apache.activemq.filter.MessageEvaluationContext;
 import org.apache.activemq.selector.SelectorParser;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 public class DestinationView implements DestinationViewMBean {
+    private static final Log log = LogFactory.getLog(DestinationViewMBean.class);
     protected final Destination destination;
     protected final ManagedRegionBroker broker;
 
@@ -130,7 +134,7 @@
                 }
                 
             } catch(Throwable e) {
-                e.printStackTrace();
+                log.warn("exception browsing destination",e);
             }
         }
         
@@ -172,7 +176,7 @@
 
             }
             catch (Throwable e) {
-                e.printStackTrace();
+                log.warn("exception browsing destination",e);
             }
         }
         return answer;
@@ -210,7 +214,7 @@
                 }
                 rc.put(new CompositeDataSupport(ct,factory.getFields(messages[i])));
             } catch(Throwable e) {
-                e.printStackTrace();
+                log.warn("exception browsing destination",e);
             }
         }