You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2009/04/15 13:01:29 UTC

svn commit: r765133 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java

Author: gtully
Date: Wed Apr 15 11:01:29 2009
New Revision: 765133

URL: http://svn.apache.org/viewvc?rev=765133&view=rev
Log:
remove duplicate log info, AMQ-2198|https://issues.apache.org/activemq/browse/AMQ-2198

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java?rev=765133&r1=765132&r2=765133&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/network/DemandForwardingBridgeSupport.java Wed Apr 15 11:01:29 2009
@@ -549,10 +549,7 @@
             // a network subscription so there is a need to synchronise on a shared entity
             synchronized(brokerService.getVmConnectorURI()) {
                 if (isDuplicateNetworkSubscription(info)) {
-                    if (LOG.isDebugEnabled()) {
-                        LOG.debug(configuration.getBrokerName() + " Ignoring sub from " + remoteBrokerName + ", destination " + info.getDestination() 
-                                + ", for " + info.getConsumerId() + " as a duplicate. Already subscribed via network subscription :"  + info);
-                    }
+                    // trace in method
                     return;
                 }
                 if (addConsumerInfo(info)) {
@@ -974,7 +971,8 @@
             if (!networkConsumers.isEmpty()) {
                 if (matchFound(candidateConsumers, networkConsumers)) {
                     if (LOG.isDebugEnabled()) {
-                        LOG.debug(configuration.getBrokerName() + " subscription: " + consumerInfo + " is duplicated by network subscription: " 
+                        LOG.debug(configuration.getBrokerName() + " Ignoring duplicate subscription from " + remoteBrokerName
+                                + ", sub: " + consumerInfo + " is duplicated by network subscription: " 
                                 + sub.getConsumerInfo()  + ", networkComsumerIds: " + networkConsumers);
                     }
                     isDuplicate = true;