jacek-lewandowski commented on PR #2672: URL: https://github.com/apache/cassandra/pull/2672#issuecomment-1709922136
Indeed I forgot about setting the delay to 0 for unit tests. Fixed now. I'll not move that method outside of the distributed "module" because it applies only to the in-jvm distributed tests, where the delay is non-zero. I'd like to not modify the delay for the distributed tests as they usually start multiple nodes and we may fall into another category of problems. Repeated tests on CircleCI makes zero sense to me in this case because: 1. the problem has not popped up on CircleCI, so why would it now? 2. I found the problem and was able to reproduce it reliably - then after applying a fix, the problem went away -- 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]
