[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_r1153766802


##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/synccontext/named/NamedLockFactoryAdapter.java:
##########
@@ -48,6 +49,14 @@ public final class NamedLockFactoryAdapter {
 
     public static final TimeUnit DEFAULT_TIME_UNIT = TimeUnit.SECONDS;
 
+    public static final String RETRIES_KEY = "aether.syncContext.named.retries";

Review Comment:
   Sure, but pls consider this PR as "under testing", this is what users on ML tested (I provided them "unofficial builds" via Google Drive): https://lists.apache.org/thread/q4nlojworr8lynroyv2j0721vpmpgh70
   
   Will add doco next.



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