[GitHub] [maven-deploy-plugin] mattnelson commented on a diff in pull request #35: [MDEPLOY-206] Support parallel deployment for deployAtEnd

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

mattnelson commented on code in PR #35:
URL: https://github.com/apache/maven-deploy-plugin/pull/35#discussion_r1092292051


##########
src/main/java/org/apache/maven/plugins/deploy/DeployMojo.java:
##########
@@ -71,6 +75,14 @@ public class DeployMojo extends AbstractDeployMojo {
     @Parameter(defaultValue = "false", property = "deployAtEnd")
     private boolean deployAtEnd;
 
+    /**
+     * The number of parallel threads which will be used for deployment with the {@code deployAtEnd} feature.
+     *
+     * @since 3.1.0
+     */
+    @Parameter(property = "maven.deploy.parallelThreads", defaultValue = "1")

Review Comment:
   sorry for the late review.
   
   Would it be possible to support dynamic syntax like is done with the CLI parallelism flag `-T 1C` that way the deploy threads could scale with the hardware without being tightly coupled to config.



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