mbaechler commented on code in PR #2903:
URL: https://github.com/apache/james-project/pull/2903#discussion_r2671920011
##########
server/protocols/webadmin-integration-test/distributed-webadmin-integration-test/src/test/java/org/apache/james/webadmin/integration/rabbitmq/ConsistencyTasksIntegrationTest.java:
##########
@@ -143,11 +142,7 @@ TestingSession
provideSession(SessionWithInitializedTablesFactory factory) {
.server(configuration ->
CassandraRabbitMQJamesServerMain.createServer(configuration)
// Enforce a single eventBus retry. Required as Current Quotas are
handled by the eventBus.
.overrideWith(binder ->
binder.bind(RetryBackoffConfiguration.class)
- .toInstance(RetryBackoffConfiguration.builder()
- .maxRetries(1)
- .firstBackoff(Duration.ofMillis(2))
- .jitterFactor(0.5)
- .build()))
+ .toInstance(RetryBackoffConfiguration.FAST))
Review Comment:
Well, it was not obvious to me because `RetryBackoffConfiguration` doesn't
refer to it and is a generic concept.
Not sure what's the best way to make it clear we are changing the EventBus
configuration here.
--
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]
