Remember artifact handlers after they've been used once

Otherwise we have to go through the whole sisu engine again, which is very slow, because it does a linear scan.
apache · Apr 10, 2019 · 571da0e · 571da0e
1 parent a48f6e4
commit 571da0e
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/...rc/main/java/org/apache/maven/artifact/handler/manager/DefaultArtifactHandlerManager.java b/...rc/main/java/org/apache/maven/artifact/handler/manager/DefaultArtifactHandlerManager.java
@@ -39,11 +39,11 @@ public class DefaultArtifactHandlerManager
     @Requirement( role = ArtifactHandler.class )
     private Map<String, ArtifactHandler> artifactHandlers;
 
-    private Map<String, ArtifactHandler> unmanagedHandlers = new ConcurrentHashMap<>();
+    private Map<String, ArtifactHandler> allHandlers = new ConcurrentHashMap<>();
 
     public ArtifactHandler getArtifactHandler( String type )
     {
-        ArtifactHandler handler = unmanagedHandlers.get( type );
+        ArtifactHandler handler = allHandlers.get( type );
 
         if ( handler == null )
         {
@@ -53,6 +53,7 @@ public ArtifactHandler getArtifactHandler( String type )
             {
                 handler = new DefaultArtifactHandler( type );
             }
+            allHandlers.put( type, handler );
         }
 
         return handler;
@@ -61,7 +62,7 @@ public ArtifactHandler getArtifactHandler( String type )
     public void addHandlers( Map<String, ArtifactHandler> handlers )
     {
         // legacy support for maven-gpg-plugin:1.0
-        unmanagedHandlers.putAll( handlers );
+        allHandlers.putAll( handlers );
     }
 
     @Deprecated