Skip to content

feat(push): parallelize push operations with goroutines#42

Merged
jamesjohnsdev merged 1 commit into
mainfrom
feat/parallel-push
Jun 21, 2026
Merged

feat(push): parallelize push operations with goroutines#42
jamesjohnsdev merged 1 commit into
mainfrom
feat/parallel-push

Conversation

@jamesjohnsdev

@jamesjohnsdev jamesjohnsdev commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Serial push was a bottleneck when many issues needed syncing.
Both (all-modified) and (new T-issues) now fan out
up to 10 concurrent gh API calls. Filter pass stays serial to avoid
concurrent isModified disk-read races.


Summary by cubic

Parallelize push operations to speed up syncing when many issues need updates. Modified and new T-numbered issues now push with up to 10 concurrent GitHub API calls, while disk-read filtering stays serial to avoid races.

  • New Features
    • push: Parallel pushes for modified issues (10-worker semaphore), with an atomic count and “Nothing to push” when zero.
    • sync: Parallel pushes for new T-numbered issues before reload, with a summary count.
    • Error handling: Collects worker errors and returns the first encountered.

Written for commit 9f9531d. Summary will update on new commits.

Review in cubic

Serial push was a bottleneck when many issues needed syncing.
Both  (all-modified) and  (new T-issues) now fan out
up to 10 concurrent gh API calls. Filter pass stays serial to avoid
concurrent isModified disk-read races.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@jamesjohnsdev jamesjohnsdev merged commit 311e4f6 into main Jun 21, 2026
6 checks passed
@jamesjohnsdev jamesjohnsdev deleted the feat/parallel-push branch June 21, 2026 10:55
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