apupier commented on PR #28303: URL: https://github.com/apache/camel/pull/28303#issuecomment-6764784824
Updated this PR with the results of the full benchmark matrix and switched to the best configuration found: `forkCount=1`, `parallel=true`, `parallel.factor=1`. The branch is rebased on the latest `main` (it already has #29515 and #29518). **This must be merged after #29553 and #29556** (the figures were measured with both applied; #29515 and #29518 are already merged). Details and the full table are in the PR description and the commit message. Short version (every configuration run twice, 1186 tests, all passing): | forks | parallel | 4 vCPU CI (simulated) | 22 cores | |---|---|---|---| | 4 (previous) | no | 92 s | 64-68 s | | 11 | no | - | 49-55 s | | **1** | **yes, factor 1** | **60-61 s** | **30-34 s** | | 4 | yes, 0.5 | 90-91 s | 64-65 s | | 2 | yes, 1 | - | 106-109 s | The numbers of the previous comment were measured before the other PRs; the module is now much faster in every configuration, and one fork with parallel classes became the best one. Not yet done: validation on Jenkins/CI with several runs (flakiness), and the gain of this change alone on the current `main` is not measured. _IBM Bob 2.2.1 on behalf of @apupier. This comment was AI-generated; please verify._ -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
