git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add support for parallel HTTP transfers

From
NHNick Hengeveld <nickh@reactrix.com>
Date
Oct 7, 2005, 17:22 UTC
Message-ID
<20051007172206.GK15593@reactrix.com>
In-Reply-To
<7v8xx55kia.fsf@assigned-by-dhcp.cox.net>
On Fri, Oct 07, 2005 at 10:01:33AM -0700, Junio C Hamano wrote:
Show 5 quoted lines
> The check was added in 029f6de377c7e0484f5c4cf070934599580f1784
> because back then calling fetch() on an object that we already
> had had a funny interaction with what http-fetch.c did.  I
> suspect that Nick's curl-multi changes made it unnecessary, but
> you should double check for other transports.

I think the only downside to leaving that check in place is that when pull() finishes there may be completed requests left behind in the queue, possibly with unreported transfer errors. Would it make sense to just release any requests left in the queue after pull(), and report if any of them had transfer errors?

-- 
For a successful technology, reality must take precedence over public
relations, for nature cannot be fooled.
Previous: Junio C HamanoNext: Junio C Hamano
Message 9 of 14 in “Add support for parallel HTTP transfers”
  1. Add support for parallel HTTP transfersNick Hengeveld, Oct 5, 2005
  2. Daniel BarkalowOct 6, 2005
  3. Nick HengeveldOct 7, 2005
  4. Junio C HamanoOct 7, 2005
  5. Nick HengeveldOct 7, 2005
  6. Junio C HamanoOct 7, 2005
  7. Daniel BarkalowOct 7, 2005
  8. Junio C HamanoOct 7, 2005
  9. Nick HengeveldOct 7, 2005
  10. Junio C HamanoOct 7, 2005
  11. Daniel BarkalowOct 7, 2005
  12. Jon LoeligerOct 10, 2005
  13. Daniel BarkalowOct 7, 2005
  14. Junio C HamanoOct 7, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.