Skip to content

Fix parallel analysis concurrency on macOS (bash 3.2)#1

Merged
yoaquim merged 1 commit into
yoaquim:mainfrom
jedibrillo:fix/parallel-analysis-concurrency
Apr 25, 2026
Merged

Fix parallel analysis concurrency on macOS (bash 3.2)#1
yoaquim merged 1 commit into
yoaquim:mainfrom
jedibrillo:fix/parallel-analysis-concurrency

Conversation

@jedibrillo
Copy link
Copy Markdown
Collaborator

Summary

  • Fix broken parallelism in analysis phase on macOS which ships with bash 3.2
  • The previous wait -n approach required bash 4.3+ and fell back to wait (waits for ALL jobs)
  • Replace with xargs -P which properly maintains N concurrent jobs, spawning a new one immediately when any finishes

Test plan

  • Run sync on macOS and observe that new analysis jobs start as soon as any job finishes (not waiting for entire batch)
  • Verify analysis output is still correct

🤖 Generated with Claude Code

The previous implementation used `wait -n` which requires bash 4.3+.
On macOS (bash 3.2), it fell back to `wait` which waits for ALL jobs,
breaking the intended parallelism (5 jobs start, all 5 must finish
before the next batch).

Replace with `xargs -P` which properly maintains N concurrent jobs,
spawning a new one immediately when any job finishes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@yoaquim yoaquim merged commit 54efa0a into yoaquim:main Apr 25, 2026
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.

2 participants