Author: gtully
Date: Tue Feb 17 11:57:23 2009
New Revision: 744975

URL: http://svn.apache.org/viewvc?rev=744975&view=rev
Log:
resolve AMQ-2121, shared lock needs to be less contentious

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=744975&r1=744974&r2=744975&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
 Tue Feb 17 11:57:23 2009
@@ -533,7 +533,7 @@
             
             // in a cyclic network there can be multiple bridges per broker 
that can propagate
             // a network subscription so there is a need to synchronise on a 
shared entity
-            synchronized(brokerService) {
+            synchronized(brokerService.getVmConnectorURI()) {
                 if (isDuplicateNetworkSubscription(info)) {
                     if (LOG.isDebugEnabled()) {
                         LOG.debug(configuration.getBrokerName() + " Ignoring 
sub from " + remoteBrokerName + ", destination " + info.getDestination() 


Reply via email to