jbertram commented on a change in pull request #3957:
URL: https://github.com/apache/activemq-artemis/pull/3957#discussion_r816036032
##########
File path:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/ClusterConnectionBridge.java
##########
@@ -118,9 +119,20 @@ public ClusterConnectionBridge(final ClusterConnection
clusterConnection,
final MessageFlowRecord flowRecord,
final TransportConfiguration connector,
final String storeAndForwardPrefix,
- final StorageManager storageManager) {
- super(targetLocator, initialConnectAttempts, reconnectAttempts, 0, //
reconnectAttemptsOnSameNode means nothing on the clustering bridge since we
always try the same
- retryInterval, retryMultiplier, maxRetryInterval, nodeUUID, name,
queue, executor, filterString, forwardingAddress, scheduledExecutor,
transformer, useDuplicateDetection, user, password, server,
ComponentConfigurationRoutingType.valueOf(ActiveMQDefaultConfiguration.getDefaultBridgeRoutingType()));
+ final StorageManager storageManager) throws
ActiveMQException {
+ super(targetLocator, new BridgeConfiguration()
+ .setInitialConnectAttempts(initialConnectAttempts)
+ .setReconnectAttempts(reconnectAttempts)
+ .setReconnectAttemptsOnSameNode(0) // reconnectAttemptsOnSameNode
means nothing on the clustering bridge since we always try the same
+ .setRetryInterval(retryInterval)
+ .setRetryIntervalMultiplier(retryMultiplier)
+ .setMaxRetryInterval(maxRetryInterval)
+ .setFilterString(filterString.toString())
Review comment:
Thanks for the heads-up, @gemmellr. I've sent a few commits directly to
`main` that should take care of the issues.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
