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

merge unpack-objects to index-pack?

From
Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date
May 11, 2012, 12:02 UTC
Message-ID
<CACsJy8BjA-L=jOZ7G9TbDY6D0rqhuzCCRwAjazEfBx_=HcqcqA@mail.gmail.com>
Hi,

I'm looking at adding large file support to unpack-objects. A simple way is just to stream large blobs into loose large blobs. But I'd rather keep those blobs in pack because pack-objects is happier that way. I'm looking at unpack-objects and thinking maybe it's best to just merge it back to index-pack.

In normal mode (all small objects), index-pack receives the pack stream. Objects will be unpacked in phase two, resolving objects in index-pack. The only downside I can see is the new unpack-objects now write temporary pack on disk, which does not sound too bad to me. unpack-packs is called on small packs so extra space is small. For single-huge-blob packs, it's good to keep them on disk anyway. When the pack has large blobs, we could either just keep full pack.

After this, the only pack receiver at client side is index-pack. fetch-pack does not have to choose between unpack-objects and index-pack, just pass --unpack-limit <n> to index-pack.

What do you think?
-- 
Duy
Next: Junio C Hamano
Message 1 of 2 in “merge unpack-objects to index-pack?”
  1. Nguyen Thai Ngoc DuyMay 11, 2012
  2. Junio C HamanoMay 11, 2012

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.