Github user gaohoward commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1984#discussion_r178696366
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/ClusterConnectionBridge.java
 ---
    @@ -127,9 +126,6 @@ public ClusterConnectionBridge(final ClusterConnection 
clusterConnection,
           this.managementNotificationAddress = managementNotificationAddress;
           this.flowRecord = flowRecord;
     
    -      // we need to disable DLQ check on the clustered bridges
    -      queue.setInternalQueue(true);
    --- End diff --
    
    @clebertsuconic no it doesn't. But it's redundant because this has already 
been set during the creation of cluster connection bridge. See
    
org.apache.activemq.artemis.core.server.cluster.impl.ClusterConnectionImpl#nodeUP()


---

Reply via email to