clebertsuconic commented on code in PR #5430:
URL: https://github.com/apache/activemq-artemis/pull/5430#discussion_r1910565512
##########
tests/artemis-test-support/src/main/java/org/apache/activemq/artemis/tests/util/ActiveMQTestBase.java:
##########
@@ -545,7 +545,7 @@ protected static final ClusterConnectionConfiguration
basicClusterConnectionConf
setName("cluster1").setAddress("jms").setConnectorName(connectorName).
setRetryInterval(100).setDuplicateDetection(false).setMaxHops(1).
setConfirmationWindowSize(1).setMessageLoadBalancingType(MessageLoadBalancingType.STRICT).
- setStaticConnectors(connectors0);
+
setStaticConnectors(connectors0).setCallTimeout(1000).setCallFailoverTimeout(1000);
Review Comment:
and think about... why would you wait 30 seconds to kill a timeout that will
certainly fail...
our test suite needs to lower the 30 seconds to 1 second (or less) for sure.
I had actually played with 500 milliseconds before I pushed the final PR
without any 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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
