Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1984#discussion_r178814438
--- 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 --
Ok.. I see..
I have a small refactoring on this.. instead of trusting queue.isInternal I
think we should move nodeUp to Bridge, and @override on the ClusterBridge.
I'm running some tests before I merge it.
---
