You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2020/06/22 07:22:45 UTC

[GitHub] [maven-resolver] rreich opened a new pull request #60: synchronize all calls to resolve on a single static lock

rreich opened a new pull request #60:
URL: https://github.com/apache/maven-resolver/pull/60


   The lock has to be static as there are multiple instances of this class.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [maven-resolver] michael-o closed pull request #60: [MRESOLVER-114] synchronize all calls to resolve on a single static lock

Posted by GitBox <gi...@apache.org>.
michael-o closed pull request #60:
URL: https://github.com/apache/maven-resolver/pull/60


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [maven-resolver] michael-o commented on pull request #60: [MRESOLVER-114] synchronize all calls to resolve on a single static lock

Posted by GitBox <gi...@apache.org>.
michael-o commented on pull request #60:
URL: https://github.com/apache/maven-resolver/pull/60#issuecomment-653803590


   I consider this approach as superseded.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [maven-resolver] michael-o commented on pull request #60: [MRESOLVER-114] synchronize all calls to resolve on a single static lock

Posted by GitBox <gi...@apache.org>.
michael-o commented on pull request #60:
URL: https://github.com/apache/maven-resolver/pull/60#issuecomment-647345704


   This will basically kill all concurrent access to non-affected artifacts. This is not a solution, but a workaround.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [maven-resolver] rreich commented on pull request #60: [MRESOLVER-114] synchronize all calls to resolve on a single static lock

Posted by GitBox <gi...@apache.org>.
rreich commented on pull request #60:
URL: https://github.com/apache/maven-resolver/pull/60#issuecomment-647352996


   I think it's all I can offer. It is not elegant I know, but I cannot see any practical downsides and I'd rather waste some milliseconds than having race conditions.


----------------------------------------------------------------
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:
users@infra.apache.org