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

Re: [RFC PATCH] packfile: iterate packed objects in pack order

From
JTJonathan Tan <jonathantanmy@google.com>
Date
Aug 9, 2018, 22:03 UTC
Message-ID
<CAGf8dgK_yMEtyh+_chQ+9i2d1Uc9oAXr7Q_8ES5OPF8qF87EHw@mail.gmail.com>
In-Reply-To
<20180808232515.GC21882@sigill.intra.peff.net>
On Wed, Aug 8, 2018 at 4:25 PM, Jeff King <peff@peff.net> wrote:
> Even if you just use the oid to do a separate lookup in the object
> database, there's still a benefit in accessing the objects in pack
> order.
You're probably right, but I don't immediately see what the benefit is.

On a not completely unrelated note, I just realized that in my patch, i should be pack_nr (ordinal in pack order) and pack_nr should be index_nr (ordinal in index order, i.e. alphabetical order). If you run with this, feel free to write your own patch and maybe I'll learn how accessing objects in pack order benefits looking up the object database through the commit message.

> I can try to pick this up and carry the cat-file bits to completion if
> you want, but probably not until tomorrow or Friday.
Thanks - feel free to do this.
Previous: Jeff KingNext: Jeff King
Message 3 of 4 in “packfile: iterate packed objects in pack order”
  1. packfile: iterate packed objects in pack orderJonathan Tan, Aug 8, 2018
  2. Jeff KingAug 8, 2018
  3. Jonathan TanAug 9, 2018
  4. Jeff KingAug 10, 2018

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.