Skip to content

Revert PR #924: Lite tests parallelization (slowed CI)#926

Merged
erikdarlingdata merged 1 commit intodevfrom
revert/lite-tests-parallel
May 2, 2026
Merged

Revert PR #924: Lite tests parallelization (slowed CI)#926
erikdarlingdata merged 1 commit intodevfrom
revert/lite-tests-parallel

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

Reverts PR #924. The parallelization config was meant to speed up the Lite test step but actually made it worse on the GH Actions Windows runner: 8m 49s vs the prior ~7-min baseline. The parallelAlgorithm: aggressive + maxParallelThreads: -1 settings cause thread-contention overhead on the runner's 2 cores. Local 2m 21s wasn't representative.

This restores the original test setup. Follow-up work (if revisited) should:

  • Set maxParallelThreads: 2 to match the runner's core count
  • Default parallelAlgorithm: conservative instead of aggressive
  • Possibly split FactCollectorTests (the slowest class at 37 tests × ~15s each) since within-class tests still run sequentially

Test plan

  • Build is green after revert
  • CI test step returns to ~7m baseline

🤖 Generated with Claude Code

…ts-parallel"

This reverts commit 987dc61, reversing
changes made to 76d825f.
@erikdarlingdata erikdarlingdata merged commit 62ef9d5 into dev May 2, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the revert/lite-tests-parallel branch May 2, 2026 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant