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

Re: [GSoC] Designing a faster index format - Progress report

From
Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date
May 27, 2012, 12:23 UTC
Message-ID
<CACsJy8D+WgEr4i2H-1oiBLY5oLurM0aNxGovbVEZDvr7OGgknw@mail.gmail.com>
In-Reply-To
<7vbolaotwj.fsf@alter.siamese.dyndns.org>
On Sun, May 27, 2012 at 4:27 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 14 quoted lines
> Thomas Gummerer <t.gummerer@gmail.com> writes:
>
>>> No, read_index_from would go through the normal tree->list conversion.
>>> What I'd like to see is what it looks like when a command accesses
>>> index v5 directly in tree form, taking all advantages that tree-form
>>> provides, and how we should deal with old index versions while still
>>> supporting index v5 (without losing tree advantages)
>>
>> Ah ok, thanks for the clarification, I understand what you meant now.
>> I think however, that it's not very beneficial to do this conversion
>> now. git ls-files needs the whole index file anyway, so it's probably
>> not a very good test.
>
> Think about "git ls-files t/" and "git ls-files -u".
Or harder things like "ls-files -- 't/*.sh'"
Show 5 quoted lines
> The former obviously does *not* have to look at the whole thing, even
> though the current code assumes the in-core data structure that has the
> whole thing in a flat array.  IIRC, you had unmerged entries tucked at the
> end outside the main index data, so the latter is also an interesting
> demonstration of how wonderful the new data format could be.

and "ls-files -uc" can show how you combine unmerged entries back. There's also entry existence check deep in "ls-files -o" that you can show how good bsearch on trees is, though that might be going too far for an experiment because the call chain is really deep, way outside ls-files.c:

show_files (builtin/ls-files.c)
 fill_directory (dir.c)
  read_directory
   read_directory_recursive
    treat_path
     treat_one_path
      treat_directory
       directory_exists_in_index
        cache_pos_name (read-cache.c)

I just want to make sure that by exercising the new format with some real problems, we are certain we don't overlook anything in designing the format (or else could be fixed before finalizing it).

-- 
Duy
Previous: Junio C HamanoNext: Thomas Gummerer
Message 9 of 13 in “[GSoC] Designing a faster index format - Progress report”
  1. Thomas GummererMay 23, 2012
  2. Thomas RastMay 24, 2012
  3. Junio C HamanoMay 24, 2012
  4. Nguyen Thai Ngoc DuyMay 25, 2012
  5. Thomas GummererMay 25, 2012
  6. Nguyen Thai Ngoc DuyMay 26, 2012
  7. Thomas GummererMay 27, 2012
  8. Junio C HamanoMay 27, 2012
  9. Nguyen Thai Ngoc DuyMay 27, 2012
  10. Thomas GummererMay 28, 2012
  11. Thomas RastMay 29, 2012
  12. Nguyen Thai Ngoc DuyMay 29, 2012
  13. Junio C HamanoMay 29, 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.