fkalinski commented on issue #310: URL: https://github.com/apache/maven/pull/310#issuecomment-617364690
I've done actually the same change and it works for me. The only difference, is not factoring the ThreadlLocals into a common class, but plainly using the three of them on the same fields (https://github.com/fkalinski/maven/commit/b38e3c2bfb6183c0f45214012112a1c1ca4048bd) Before the change, parallel builds were failing around 30% of the time, on the 64-module project I work on. Due to many inter-dependencies, maximum parallelism than can be achieved on the project is around 4. When build fails. it fails fast, as the initial modules that are built are of a small size. After the change applied again today, I haven't seen a failed build, yet. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
