rmannibucau commented on pull request #65: URL: https://github.com/apache/maven-resolver/pull/65#issuecomment-661280048
2. Syncontext must be global if you want an useful global lock but it uses an int so it is not thread safe. If not global it does not impl its original goal so in all cases it looks incomplete. 3. Looked at github bugtracker as you requested and it does not look terrible and not worse that this PR was to me. Do you have particular links? Also the point to enable to lock per artifact is not bound to these impl issues, the impl still proves there is no mem issue to do it (was a point against it which is not justified technically so wanted to ensure we mark this topic as done). ---------------------------------------------------------------- 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]
