quantranhong1999 commented on PR #1635: URL: https://github.com/apache/james-project/pull/1635#issuecomment-1623756815
>Instead I would write a single test proving delays not to work in distributed mode: IMO your suggestion just proves something (delay send on Distributed) **could not** work. It does not change the fact that the current contract test suite is not strong enough to prove something (delay send on whatever platform) **could** work. Thanh tried to run the test suite with James Memory with delay send **disabled**, and delay send tests are still passed... What if in the future we have a JMAP server on top of the Pulsar mail queue, how do we make sure the delay send **could** work? Exchanging 4 seconds of build time to get more reliable tests is a good trade IMO. -- 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]
