From: Nick Edelen <sirnot@gmail.com>
To: Junio C Hamano <gitster@pobox.com>, Nicolas Pitre <nico@cam.org>,
Johannes Schindelin <Johannes.Schindelin@gmx.de>,
Sam Vilain <sam@vilain.net>,
Michael J Gruber <git@drmicha.warpmail
Subject: [PATCH 0/7 (v5)] rev-cache
Date: Mon, 05 Apr 2010 20:57:43 +0100 [thread overview]
Message-ID: <4BBA40B7.1060100@gmail.com> (raw)
SUGGESTED FOR 'PU':
Traversing objects is currently very costly, as every commit and tree must be
loaded and parsed. Much time and energy could be saved by caching metadata and
topological info in an efficient, easily accessible manner. Furthermore, this
could improve git's interfacing potential, by providing a condensed summary of
a repository's commit tree.
This is a series to implement such a revision caching mechanism, aptly named
rev-cache. The series will provide:
- a core API to manipulate and traverse caches
- an integration into the internal revision walker
- a porcelain front-end providing access to users and (shell) applications
- a series of tests to verify/demonstrate correctness
- documentation of the API, porcelain and core concepts
In cold starts rev-cache has sped up packing and walking by a factor of 4, and
over twice that on warm starts. The mechanism is minimally intrusive: most of
the changes take place in seperate files, and only a handful of git's existing
functions are modified.
Slides from the presentation I gave at GitTogether'09 give a good overview of
the mechanism, usage and performance of rev-cache. (well, they complement the
documentation well. they're a bit bare without the dialogue...) You can find
them at http://www.flickr.com/photos/sirnot/sets/72157623652754819/
The patchset is structured so that each patchfile represents a fully
functional, incremental development of rev-cache. Unfortunately the addition
of several files makes for some obscenely large patchfiles. I apologize and
regret that this is somewhat unavoidable; my only consolation is that
patchfiles are less 'patches' and more additions of files.
Hope you find this useful.
- Nick
---
Hi everyone. I'm sorry about the (very) belated patchset -- University has a
tendency of being very absorbing. It's spring break now though, so I've
finally got around to finalizing rev-cache! The only significant change from
last time (besides the bugfixes) is how structures are read/written --
excessive care is now taken to ensure a standardized file format.
This time round I've also got some swanky slides I made for GitTogether. Hehe.
Documentation/git-rev-cache.txt | 194 +++
Documentation/technical/rev-cache.txt | 634 +++++++++
Makefile | 2 +
builtin.h | 1 +
builtin/gc.c | 9 +
builtin/rev-cache.c | 339 +++++
command-list.txt | 1 +
commit.c | 36 +-
git.c | 1 +
list-objects.c | 46 +-
object.h | 3 +-
rev-cache.c | 2468 +++++++++++++++++++++++++++++++++
rev-cache.h | 123 ++
revision.c | 90 +-
revision.h | 44 +-
t/t6019-rev-cache-list.sh | 263 ++++
16 files changed, 4227 insertions(+), 27 deletions(-)
create mode 100644 Documentation/git-rev-cache.txt
create mode 100644 Documentation/technical/rev-cache.txt
create mode 100644 builtin/rev-cache.c
create mode 100644 rev-cache.c
create mode 100644 rev-cache.h
create mode 100644 t/t6019-rev-cache-list.sh
next reply other threads:[~2010-04-05 19:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-05 19:57 Nick Edelen [this message]
[not found] ` <7vaathjcru.fsf@alter.siamese.dyndns.org>
2010-04-06 10:14 ` [PATCH 0/7 (v5)] rev-cache Björn Gustavsson
2010-04-06 17:34 ` Junio C Hamano
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=4BBA40B7.1060100@gmail.com \
--to=sirnot@gmail.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@drmicha.warpmail \
--cc=gitster@pobox.com \
--cc=nico@cam.org \
--cc=sam@vilain.net \
/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).