CAMEL-24318 - Use Maven parallelization for deploy on Jenkins - #25264
Conversation
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
* locally with local repo (take care to deactivate local build cache): * 8'32" ( ./mvnw -T1C -U -B -e -fae -V -Dnoassembly -Pdeploy,apache-snapshots -Dquickly clean deploy -DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2 ) * 18'33" (./mvnw -U -B -e -fae -V -Dnoassembly -Pdeploy,apache-snapshots -Dquickly clean deploy -DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2) * on jenkins for Camel core: * 25min instead of usually 3 hours, previous record was 2h35 , see https://ci-builds.apache.org/job/Camel/job/Camel%20Daily%20Snapshot%20Deploy/job/main/1494/ Signed-off-by: Aurélien Pupier <apupier@ibm.com>
1a6b1bd to
634185e
Compare
gnodet
left a comment
There was a problem hiding this comment.
Claude Code on behalf of Guillaume Nodet
Adding -T1C to the snapshot deploy pipeline is a well-tested, low-risk optimization. The deploy pipeline uses -Dquickly (skipping tests/formatting/enforcement), and all plugins involved (compiler, deploy, source, javadoc) are thread-safe. The PR provides good performance evidence — local measurements (8m32s vs 18m33s) and a Jenkins build link (build #1494 at 25min vs the typical ~3h baseline).
Key observations:
- The main
Jenkinsfile(full build with tests) is correctly left unchanged — full builds with test execution carry higher thread-safety risk - The existing
-fae(fail-at-end) flag is the correct companion for-T1C - Maven reactor properly respects inter-module dependency ordering even with
-T1C - Memory consumption may increase with parallel builds since multiple modules compile and generate javadoc simultaneously — worth monitoring after merge, though the Jenkins test (build #1494) demonstrates sufficient resources
-DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2 )
-Pdeploy,apache-snapshots -Dquickly clean deploy
-DaltDeploymentRepository=your_repo_id::default::file:/tmp/alt-repo-2)
Description
Target
mainbranch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTestslocally from root folder and I have committed all auto-generated changes.AI-assisted contributions
Co-authored-bytrailers) and the PR description identifies the AI tool used.