kgyrtkirk commented on PR #17861: URL: https://github.com/apache/druid/pull/17861#issuecomment-2930895709
I've opened a PR with relevant fixes: https://github.com/apache/druid/pull/18062 I've improved the re-usage of some parts - which seem to have improved the execution time from 38m to 28m - not sure if that's really the case..we will see it in CI runs if that's really the case. but still a lot of threads were created so I was also digging into for the reason why there were so many extra threads were created - it plays a role that we have a lot of different suppliers (71) allocating some extra threads...not sure what to do with it right now - maybe its fine...but going down would require the system to spend less time initializing stuff.... ``` git grep 'use druidtest'|cut -d: -f 2-|sort|uniq -c ``` I'll keep thinking about it. if you encounter such failures in the future - the `iq` file doesn't necessarily contain the most relevant exceptions; its just the one which surfaces ; like when you are using the console - however there is an xml file at `quidem-ut/target/surefire-reports/TEST-org.apache.druid.quidem.QTest.xml` which usually contains further details -- 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]
