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

Run downloads in parallel #32

Open
wants to merge 1 commit into
base: live
Choose a base branch
from
Open

Run downloads in parallel #32

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 19, 2024

No description provided.

@ghost ghost requested a review from Bjwebb February 19, 2024 07:24
@ghost ghost self-assigned this Feb 19, 2024
@ghost ghost force-pushed the 2024-faster branch from 1ac4afa to f2d4d16 Compare February 23, 2024 17:31
@Bjwebb
Copy link
Contributor

Bjwebb commented Feb 23, 2024

Can we set this up on a dev server, and do a check whether more downloads are blocked compared to live? It would also be good to know whether anything's missing from https://iati-data-dump.codeforiati.org/ for probably this reason.

@andylolz
Copy link
Contributor

andylolz commented Feb 25, 2024

Can we set this up on a dev server, and do a check whether more downloads are blocked compared to live?

I feel like I checked this when switching to parallel downloads on iati-data-dump, and it wasn’t the case (though I’d be interested to hear the results of your experiments!) Also, this parallel downloading was copied from d-portal (as mentioned in the acknowledgements), so I guess they’d also have similar problems.

@ghost ghost force-pushed the 2024-faster branch from 722fae6 to 6c5dc25 Compare March 15, 2024 09:29
@ghost
Copy link
Author

ghost commented Mar 15, 2024

Thanks both. I've added --retry-connrefused and put this on dev servers.

@ghost ghost force-pushed the 2024-faster branch from 6c5dc25 to 67dfe8c Compare March 21, 2024 17:18
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.

3 participants