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

Look at allowing parallel downloads #116

Open
jrburke opened this issue Dec 5, 2012 · 1 comment
Open

Look at allowing parallel downloads #116

jrburke opened this issue Dec 5, 2012 · 1 comment
Milestone

Comments

@jrburke
Copy link
Member

jrburke commented Dec 5, 2012

In #112, @guybedford suggested doing some parallel downloading for added speed. The relavent code is in add.js around line 89. Perhaps put the promises in an array, then use q.all() as the return value from that section.

Ideally this would use a connection pool too, we do not want to flood github with a bunch of requests at once. So, the above section should add the promises to a connection queue, and the connection queue would work through them, only doing X number at a time, say 4 for now.

@rafi
Copy link

rafi commented Jul 3, 2013

Much awaited.

@jrburke jrburke modified the milestones: 0.4.0, 0.3.2 May 28, 2014
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

No branches or pull requests

2 participants