[PATCH] repack: add -F option that passes --no-reuse-delta to pack-objects
From: Jan Krüger <hidden>
Date: 2016-06-15 22:49:38
Subsystem:
documentation, the rest · Maintainers:
Jonathan Corbet, Linus Torvalds
In 479b56ba ('make "repack -f" imply "pack-objects --no-reuse-object"'),
git repack -f was changed to include recompressing all objects on the
zlib level on the assumption that if the user wants to spend that much
time already, some more time won't hurt (and recompressing is useful if
the user changed the zlib compression level).
However, "some more time" can be quite long with very big repositories,
so some users are going to appreciate being able to choose. Hence, this
adds a new -F option that uses the old behaviour of recalculating deltas
only and keeping the zlib compression intact.
Measurements taken using this patch on a current clone of git.git
indicate a 17% decrease in time being made available to users:
git repack -Adf 38.79s user 0.56s system 133% cpu 29.394 total
git repack -AdF 34.84s user 0.56s system 145% cpu 24.388 total
Signed-off-by: Jan Krüger <redacted>
---
The concrete case that prompted me to write this patch was a repository
of 25 GB that some guys were trying to repack. 17% of the time needed to
repack -f that much data is... substantial.
Discussion point: it might make more sense to switch the meanings
around, making -F do the 'bigger' routine and reverting -f to what it
used to be. I don't feel strongly about that, however.
Cc'ing those who were involved in the discussion leading to 479b56ba.
Documentation/git-repack.txt | 6 +++++-
git-repack.sh | 2 ++
2 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
index 8c67d17..cce32e2 100644
--- a/Documentation/git-repack.txt
+++ b/Documentation/git-repack.txt@@ -8,7 +8,7 @@ git-repack - Pack unpacked objects in a repository SYNOPSIS -------- -'git repack' [-a] [-A] [-d] [-f] [-l] [-n] [-q] [--window=N] [--depth=N] +'git repack' [-a] [-A] [-d] [-f] [-F] [-l] [-n] [-q] [--window=N] [--depth=N] DESCRIPTION -----------
@@ -65,6 +65,10 @@ other objects in that pack they already have locally. Pass the `--no-reuse-object` option to `git-pack-objects`, see linkgit:git-pack-objects[1]. +-F:: + Pass the `--no-reuse-delta` option to `git-pack-objects`, see + linkgit:git-pack-objects[1]. + -q:: Pass the `-q` option to 'git pack-objects'. See linkgit:git-pack-objects[1].
diff --git a/git-repack.sh b/git-repack.sh
index 1eb3bca..a010406 100755
--- a/git-repack.sh
+++ b/git-repack.sh@@ -11,6 +11,7 @@ a pack everything in a single pack A same as -a, and turn unreachable objects loose d remove redundant packs, and run git-prune-packed f pass --no-reuse-object to git-pack-objects +F pass --no-reuse-delta to git-pack-objects n do not run git-update-server-info q,quiet be quiet l pass --local to git-pack-objects
@@ -35,6 +36,7 @@ do -d) remove_redundant=t ;; -q) GIT_QUIET=t ;; -f) no_reuse=--no-reuse-object ;; + -F) no_reuse=--no-reuse-delta ;; -l) local=--local ;; --max-pack-size|--window|--window-memory|--depth) extra="$extra $1=$2"; shift ;;
--
1.7.2.3.392.g02377.dirty