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

Re: [PATCH 3/3] revision: insert unsorted, then sort in prepare_revision_walk()

From
Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date
Apr 3, 2012, 03:44 UTC
Message-ID
<CACsJy8CSohtWUV_BT-d+tGX9R4LUr1K=jgFh841nST01QLSGuA@mail.gmail.com>
In-Reply-To
<CAJo=hJsprQtjDChtrSMcne+OCeUx=NVxLHs3k_qnYLzO=aQWuw@mail.gmail.com>
On Mon, Apr 2, 2012 at 11:51 PM, Shawn Pearce <spearce@spearce.org> wrote:
Show 27 quoted lines
> On Mon, Apr 2, 2012 at 09:49, Martin Fick <mfick@codeaurora.org> wrote:
>> On Monday, April 02, 2012 10:39:59 am Shawn Pearce wrote:
>>> On Mon, Apr 2, 2012 at 09:24, Martin Fick
>> <mfick@codeaurora.org> wrote:
>>> > On Saturday, March 31, 2012 04:11:01 pm René Scharfe
>> wrote:
>>> Git can't really do the same thing as "cache the
>>> RevWalk". Its spawning a new process that needs to
>>> decompress and parse each commit object to determine its
>>> timestamp so the commits can be sorted into the priority
>>> queue. This is still an O(N) operation given N
>>> references.
>>
>> While I suspect this has been suggested before, an ondisk
>> cache of commits to timestamps would probably help here with
>> large repos.  Such a cache could make even new processes
>> able to create this list much quicker.  Since this cache
>> would contain immutable data, even if it is out of date it
>> would likely provided significant improvements by providing
>> most of the timestamps leaving only a few to parse from
>> newer commits?
>
> Probably. But we tend to hate caches in Git because they can get stale
> and need to be rebuilt, and are redundant with the base data. The
> mythical "pack v4" work was going to approach this problem by storing
> the commit timestamps uncompressed in a more machine friendly format.
> Unfortunately the work has been stalled for years.
which reminds me, hello Nico!
On Sat, Feb 18, 2012 at 10:34 PM, Nicolas Pitre <nico@fluxnic.net> wrote:
Show 9 quoted lines
>> By the way, is latest packv4 code available somewhere to fetch?
>
> Well, not yet.  Incidentally, I'm going in the Caribbeans for a week in
> a week, with no kids and only my wife who is going to be busy with scuba
> diving activities.  Like I did last year, I'm going to take some time to
> pursue my work on Pack v4 during that time.  And I intend to publish it
> when I come back, whatever state it is in, so someone else can complete
> the work eventually (I have too much to do to spend significant time on
> Git these days).

How's the packv4 work going? Is it in a public place? I hope somebody else may have spare time and be motivated enough to finish it.

-- 
Duy
Previous: Nguyen Thai Ngoc DuyNext: Jeff King
Message 33 of 37 in “Git push performance problems with ~100K refs”
  1. Martin FickMar 30, 2012
  2. Junio C HamanoMar 30, 2012
  3. Martin FickMar 30, 2012
  4. Jeff KingMar 30, 2012
  5. Jeff KingMar 30, 2012
  6. Martin FickMar 30, 2012
  7. 1/3 add mergesort() for linked listsRené Scharfe, Mar 31, 2012
  8. Junio C HamanoApr 5, 2012
  9. René ScharfeApr 8, 2012
  10. Junio C HamanoApr 9, 2012
  11. Stephen BoydApr 11, 2012
  12. Junio C HamanoApr 11, 2012
  13. 2/3 commit: use mergesort() in commit_list_sort_by_date()René Scharfe, Mar 31, 2012
  14. 3/3 revision: insert unsorted, then sort in prepare_revision_walk()René Scharfe, Mar 31, 2012
  15. Martin FickMar 31, 2012
  16. Junio C HamanoMar 31, 2012
  17. Martin FickApr 2, 2012
  18. Shawn PearceApr 2, 2012
  19. Martin FickApr 2, 2012
  20. Shawn PearceApr 2, 2012
  21. Jeff KingApr 2, 2012
  22. Jeff KingApr 2, 2012
  23. Martin FickApr 2, 2012
  24. Nguyen Thai Ngoc DuyApr 3, 2012
  25. Martin FickApr 3, 2012
  26. 0/3 Commit cacheNguyễn Thái Ngọc Duy, Apr 3, 2012
  27. 1/3 parse_commit_buffer: rename a confusing variable nameNguyễn Thái Ngọc Duy, Apr 3, 2012
  28. 2/3 Add commit cache to help speed up commit traversalNguyễn Thái Ngọc Duy, Apr 3, 2012
  29. 3/3 Add parse_commit_for_rev() to take advantage of sha1-cacheNguyễn Thái Ngọc Duy, Apr 3, 2012
  30. Nguyen Thai Ngoc DuyApr 5, 2012
  31. Shawn PearceApr 6, 2012
  32. Nguyen Thai Ngoc DuyApr 7, 2012
  33. Nguyen Thai Ngoc DuyApr 3, 2012
  34. Jeff KingApr 2, 2012
  35. René ScharfeApr 2, 2012
  36. Jeff KingApr 3, 2012
  37. Jeff KingApr 3, 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.