elharo opened a new pull request, #180:
URL: https://github.com/apache/maven-toolchains-plugin/pull/180

   Fixes #174
   
   ## Problem
   
   The static  object and  block around  are unnecessary:
   
   1. The plugin is marked , meaning Maven may execute it concurrently for 
different modules/sessions
   2.  stores data into the build context, which is already per-session and 
thread-safe
   3. The static lock introduces unnecessary contention: one module's toolchain 
selection blocks another unrelated module's selection, defeating the purpose of 
   
   ## Impact
   
   In multi-module Maven builds where the toolchains plugin runs in multiple 
modules (e.g., via inherited plugin configuration), the static lock serializes 
what could otherwise run in parallel, slowing down builds.
   
   ## Fix
   
   Remove the  field and the  block around the  call, since it provides no 
benefit.


-- 
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]

Reply via email to