Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add splitting into destination chunks #6

Merged
merged 1 commit into from
Dec 21, 2024
Merged

Conversation

dfsnow
Copy link
Owner

@dfsnow dfsnow commented Dec 19, 2024

GitHub jobs are currently partitioned into origin chunks i.e. for this job, route from 500 origins (out of say 1500 for example) to all destinations. However, some very large states have a lot of destinations, so even chunking by origin isn't guaranteed to allow a job to finish in the maximum 6 hours. Chunking by destination fixes that issue by limiting the set of destinations from each origin chunk.

GitHub jobs are currently partitioned into origin chunks i.e. for this
job, route from 500 origins (out of say 1500 for example) to all
destinations. However, some very large states have a lot of
destinations, so even chunking by origin isn't guaranteed to allow a job
to finish in the maximum 6 hours. Chunking by destination fixes that
issue by limiting the set of destinations from each origin chunk.
@dfsnow dfsnow merged commit 8e7086e into main Dec 21, 2024
28 checks passed
@dfsnow dfsnow deleted the dfsnow/add-dest-splits branch December 21, 2024 05:49
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