[GitHub] [maven-resolver] cstamas commented on a diff in pull request #272: [MRESOLVER-346] Too eager locking in resolver

["cstamas (via GitHub)" <gi...@apache.org>]

cstamas commented on code in PR #272:
URL: https://github.com/apache/maven-resolver/pull/272#discussion_r1158050479


##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultDeployer.java:
##########
@@ -197,7 +197,7 @@ public DeployResult deploy(RepositorySystemSession session, DeployRequest reques
                     e);
         }
 
-        try (SyncContext syncContext = syncContextFactory.newInstance(session, false)) {
+        try (SyncContext syncContext = syncContextFactory.newInstance(session, true)) {

Review Comment:
   yes, local repo is NOT modified, everything happens on the fly and only remote repo gets content changes



-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org