imply-cheddar opened a new pull request, #14487: URL: https://github.com/apache/druid/pull/14487
`GuavaUtilsTest.testCancelAll` is currently spinning up 3 threads, putting them in an infinite loop and then only causing one to exit. This means that 2 threads are continuing to run in the background for the entire lifetime of the surefire run just doing their infinite loop thing. Either way, the infinite loop is a poor way of doing the concurrent validation anyway, so switch it to do a proper validation with countdown latches and tasks that always exit. -- 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]
