Run tests in parallel by default - #22939
Conversation
|
Not an expert in conflicts file. Should cc @bukka in this. |
FWIW, the doc change is only a clarification because after this tests run in parallel by default. This PR has no changes to |
arnaud-lb
left a comment
There was a problem hiding this comment.
Looks good to me otherwise, but I will wait for other reviews
06569f4 to
9fb7077
Compare
0374c74 to
472acd1
Compare
arnaud-lb
left a comment
There was a problem hiding this comment.
Looks good to me otherwise!
I will wait a few days and merge if there are no objections.
9e8de6b to
5a874af
Compare
Friendly reminder, @arnaud-lb. 🫡 If this is merged #22995 would have a slight change. The same is true for #22946, so if you would merge here it would be nice if you could look at the Windows one-liner along with it. I plan to later today push the update for the CI PR. Monday is a good day to merge, isn't it? 🤓 Will be around to have an eye on CI in case anything unexpected happens! Edit: does this need a NEWS or INTERNALS entry? |
|
Yes, this is worth a NEWS and INTERNALS entry. Please add them, and I will merge :) |
That's done! 🫡 |
|
Thank you! |
Extracted from #22917.
Runs the test suite in parallel by default; automatically selects up to 10 workers. Explicit
-j1remains the opt-out to run tests serial. Smaller batches improve balancing.Intended to merge after #22957 so eligible Windows test subprocesses no longer use the shell wrapper when parallel execution becomes the default.
Result
Values in seconds; measured on macOS.
(none)Reproduction:
Ref: 9b36b91 (review comments)
cc @arnaud-lb