From: Thomas Gummerer <t.gummerer@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org, mhagger@alum.mit.edu, pclouds@gmail.com,
trast@student.ethz.ch
Subject: Re: [GSoC] Designing a faster index format - Progress report week 13
Date: Tue, 17 Jul 2012 10:24:52 +0200 [thread overview]
Message-ID: <20120717082452.GC1849@tgummerer.surfnet.iacbox> (raw)
In-Reply-To: <7vwr23zb65.fsf@alter.siamese.dyndns.org>
Thanks Junio for reading the progress report, this is just
corrected version without the errors that he pointed out.
== Work done in the previous 12 weeks ==
- Definition of a tentative index file v5 format [1]. This differs
from the proposal in making it possible to bisect the directory
entries and file entries, to do a binary search. The exact bits
for each section were also defined. To further compress the index,
along with prefix compression, the stat data is hashed, since
it's only used for equality comparison, but the plain data is
never used.
Thanks to Michael Haggerty, Nguyen Thai Ngoc Duy, Thomas Rast
and Robin Rosenberg for feedback.
- Prototype of a converter from the index format v2/v3 to the index
format v5. [2] The converter reads the index from a git repository,
can output parts of the index (header, index entries as in
git ls-files --debug, cache tree as in test-dump-cache-tree, or
the reuc data). Then it writes the v5 index file format to
.git/index-v5. Thanks to Michael Haggerty for the code review.
- Prototype of a reader for the new index file format. [3] The
reader has mainly the purpose to show the algorithm used to read
the index lexicographically sorted after the full name which is
required by the current internal memory format. Big thanks for
reviewing this code and giving me advice on refactoring goes
to Michael Haggerty.
- Read the on-disk index file format and translate it to the current
in memory format. This doesn't include reading any of the current
extensions, which are now part of the main index. The code again
is on github. [4] Thanks for reviewing the first steps to Thomas
Rast.
- Read the cache-tree data (formerly an extension, now it's integrated
with the rest of the directory data) from the new ondisk format.
There are still a few optimizations to do in this algorithm.
- Started implementing the API (suggested by Duy), but it's still
in the very early stages. There is one commit for this on GitHub [1],
but it's a very early work in progress.
- Started implementing the writer, which extracts the directories from
the in-memory format, and writes the header and the directories to
disk. The algorithm uses a hash-table instead of a simple list,
to avoid many corner cases.
- Implemented writing the file block to disk, and basic tests from the
test suite are running fine, not including tests that require
conflicted data or the cache-tree to work, which both are not
implemented yet.
- Started implementing a patch to introduce a ce_namelen field in
struct cache_entry and drop the name length from the flags. [5]
== Work done int the last week ==
- Polished the patch for the ce_namelen field. The thread for the
patch can be found at [5]. Again thanks to Junio, Duy and Thomas
for reviewing it and giving me suggestions for improving it.
- Implemented the cache-tree and conflict data writing to the
index-v5 file.
== Outlook for the next week ==
- There are still a few bugs in the conflict writing, which will
be fixed, to make the test suite pass with index-v5.
- Once the test suite passes, the code still needs to be refactored
and optimized.
- If the two points above go well, I'll continue working on the api
that Duy suggested.
[1] https://github.com/tgummerer/git/wiki/Index-file-format-v5
[2] https://github.com/tgummerer/git/blob/pythonprototype/git-convert-index.py
[3] https://github.com/tgummerer/git/blob/pythonprototype/git-read-index-v5.py
[4] https://github.com/tgummerer/git/tree/index-v5
[5] http://thread.gmane.org/gmane.comp.version-control.git/200997
next prev parent reply other threads:[~2012-07-17 8:25 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-16 20:33 [GSoC] Designing a faster index format - Progress report week 13 Thomas Gummerer
2012-07-16 20:46 ` Junio C Hamano
2012-07-17 8:21 ` Thomas Gummerer
2012-07-17 8:24 ` Thomas Gummerer [this message]
2012-07-22 15:22 ` Robin Rosenberg
2012-07-22 18:52 ` Junio C Hamano
2012-07-22 19:43 ` Thomas Rast
2012-07-22 21:08 ` Junio C Hamano
2012-07-23 22:29 ` Robin Rosenberg
2012-07-24 11:54 ` Thomas Rast
2012-07-24 16:58 ` Junio C Hamano
2012-07-25 6:44 ` Thomas Rast
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120717082452.GC1849@tgummerer.surfnet.iacbox \
--to=t.gummerer@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=mhagger@alum.mit.edu \
--cc=pclouds@gmail.com \
--cc=trast@student.ethz.ch \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).