famod opened a new pull request #413:
URL: https://github.com/apache/maven/pull/413


   Avoids concurrency issues when aggregator plugins are involved.
   
   Might also fix:
   - https://issues.apache.org/jira/browse/MNG-4996
   - https://issues.apache.org/jira/browse/MNG-5750
   - https://issues.apache.org/jira/browse/MNG-5960
   
   This is an alternative to #310 in which @rfscholte raised concerns whether 
cloning is the right approach.
   
   I went with a wrapper and a single `ThreadLocal` because mutliple TLs would 
bloat the code and would theoretically increase overhead (haven't measured it, 
though).
   I had to be creative with removing empty lines because Checkstyle was 
complaining about the class exceeding 2000 lines.
   
   PS: I removed the checklist to avoid noise (checked everything, ICLA is 
present).


----------------------------------------------------------------
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:
[email protected]


Reply via email to