Incubating Maven resolver: do not load the workspace when the origina…

…l resolver isn't aware of it, switch to choose between blocking and non-blocking dependency processor, clean-up

independent-projects/bootstrap/maven-resolver/src/main/java/io/quarkus/bootstrap/resolver/maven/ApplicationDependencyTreeResolver.java

@@ -61,6 +61,7 @@
 import io.quarkus.maven.dependency.DependencyFlags;
 import io.quarkus.maven.dependency.ResolvedDependencyBuilder;
 import io.quarkus.paths.PathTree;
+import io.quarkus.paths.PathVisit;
 
 public class ApplicationDependencyTreeResolver {
 
@@ -583,23 +584,30 @@ private ExtensionInfo getExtensionInfoOrNull(Artifact artifact, List<RemoteRepos
 
         artifact = resolve(artifact, repos);
         final Path path = artifact.getFile().toPath();
-        final Properties descriptor = PathTree.ofDirectoryOrArchive(path).apply(BootstrapConstants.DESCRIPTOR_PATH, visit -> {
-            if (visit == null) {
-                return null;
-            }
-            try {
-                return readDescriptor(visit.getPath());
-            } catch (IOException e) {
-                throw new UncheckedIOException(e);
-            }
-        });
+        final Properties descriptor = PathTree.ofDirectoryOrArchive(path).apply(BootstrapConstants.DESCRIPTOR_PATH,
+                ApplicationDependencyTreeResolver::readExtensionProperties);
         if (descriptor != null) {
             ext = new ExtensionInfo(artifact, descriptor, devMode);
             allExtensions.put(extKey, ext);
         }
         return ext;
     }
 
+    private static Properties readExtensionProperties(PathVisit visit) {
+        if (visit == null) {
+            return null;
+        }
+        try {
+            final Properties rtProps = new Properties();
+            try (BufferedReader reader = Files.newBufferedReader(visit.getPath())) {
+                rtProps.load(reader);
+            }
+            return rtProps;
+        } catch (IOException e) {
+            throw new UncheckedIOException(e);
+        }
+    }
+
     private void injectDeploymentDependencies(ExtensionDependency extDep)
             throws BootstrapDependencyProcessingException {
         log.debugf("Injecting deployment dependency %s", extDep.info.deploymentArtifact);
@@ -719,14 +727,6 @@ private void clearWalkingFlag(byte flag) {
         }
     }
 
-    private static Properties readDescriptor(Path path) throws IOException {
-        final Properties rtProps = new Properties();
-        try (BufferedReader reader = Files.newBufferedReader(path)) {
-            rtProps.load(reader);
-        }
-        return rtProps;
-    }
-
     private static class ExtensionDependency {
 
         static ExtensionDependency get(DependencyNode node) {

independent-projects/bootstrap/maven-resolver/src/main/java/io/quarkus/bootstrap/resolver/maven/BlockingModelResolutionTaskRunner.java

@@ -0,0 +1,22 @@
+package io.quarkus.bootstrap.resolver.maven;
+
+/**
+ * Blocking implementation of the {@link ModelResolutionTaskRunner}.
+ * Each call to {@link #run(ModelResolutionTask)} will block until the task has finished running.
+ * {@link #waitForCompletion()} is a no-op in this implementation.
+ */
+class BlockingModelResolutionTaskRunner implements ModelResolutionTaskRunner {
+
+    static BlockingModelResolutionTaskRunner getInstance() {
+        return new BlockingModelResolutionTaskRunner();
+    }
+
+    @Override
+    public void run(ModelResolutionTask task) {
+        task.run();
+    }
+
+    @Override
+    public void waitForCompletion() {
+    }
+}