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

download multiple packages in parallel in download-sequence #335

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

dhellmann
Copy link
Member

We use a build-order file to know what to
download, but we don't have to wait for one
download to complete to start the next one. Use
threads to download several packages in parallel.
Set a limit, but let the user control the limit
with a command line option.

@dhellmann dhellmann requested review from tiran and shubhbapna August 16, 2024 20:10
@dhellmann dhellmann added the enhancement New feature or request label Aug 16, 2024
Copy link
Collaborator

@shubhbapna shubhbapna left a comment

Choose a reason for hiding this comment

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

This will be a nice improvement! Just a couple of small comments

@dhellmann dhellmann force-pushed the multi-threaded-download branch 3 times, most recently from 310ef4b to 27407f8 Compare August 16, 2024 21:04
@mergify mergify bot added the ci label Aug 16, 2024
We use a build-order file to know what to
download, but we don't have to wait for one
download to complete to start the next one. Use
threads to download several packages in parallel.
Set a limit, but let the user control the limit
with a command line option.
@dhellmann dhellmann force-pushed the multi-threaded-download branch from 27407f8 to e84a362 Compare August 16, 2024 21:07
@mergify mergify bot merged commit fed96ea into python-wheel-build:main Aug 16, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants