cstamas commented on a change in pull request #77: URL: https://github.com/apache/maven-resolver/pull/77#discussion_r568863257
########## File path: maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/synccontext/NoLockSyncContextFactory.java ########## @@ -22,39 +22,42 @@ import java.util.Collection; import javax.inject.Named; +import javax.inject.Singleton; import org.eclipse.aether.RepositorySystemSession; import org.eclipse.aether.SyncContext; import org.eclipse.aether.artifact.Artifact; -import org.eclipse.aether.spi.synccontext.SyncContextFactory; import org.eclipse.aether.metadata.Metadata; /** * A factory to create synchronization contexts. This default implementation does not provide any real * synchronization but merely completes the repository system. */ -@Named -public class DefaultSyncContextFactory - implements SyncContextFactory, org.eclipse.aether.impl.SyncContextFactory +@Singleton +@Named( NoLockSyncContextFactory.NAME ) +public class NoLockSyncContextFactory Review comment: Frankly, I'd drop this class, as current master (and 1.6.x as for that matter) suffers from MRESOVER-153, so this class is really an implementation "we know to cause problems". ---------------------------------------------------------------- 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: us...@infra.apache.org