undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

pack-objects: default to writing bitmap hash-cache

Browse Source 
Enabling pack.writebitmaphashcache should always be a performance win.
It costs only 4 bytes per object on disk, and the timings in ae4f07fbcc
(pack-bitmap: implement optional name_hash cache, 2013-12-21) show it
improving fetch and partial-bitmap clone times by 40-50%.

The only reason we didn't enable it by default at the time is that early
versions of JGit's bitmap reader complained about the presence of
optional header bits it didn't understand. But that was changed in
JGit's d2fa3987a (Use bitcheck to check for presence of OPT_FULL option,
2013-10-30), which made it into JGit v3.5.0 in late 2014.

So let's turn this option on by default. It's backwards-compatible with
all versions of Git, and if you are also using JGit on the same
repository, you'd only run into problems using a version that's almost 5
years old.

We'll drop the manual setting from all of our test scripts, including
perf tests. This isn't strictly necessary, but it has two advantages:

  1. If the hash-cache ever stops being enabled by default, our perf
     regression tests will notice.

  2. We can use the modified perf tests to show off the behavior of an
     otherwise unconfigured repo, as shown below.

These are the results of a few of a perf tests against linux.git that
showed interesting results. You can see the expected speedup in 5310.4,
which was noted in ae4f07fbcc. Curiously, 5310.8 did not improve (and
actually got slower), despite seeing the opposite in ae4f07fbcc.
I don't have an explanation for that.

The tests from p5311 did not exist back then, but do show improvements
(a smaller pack due to better deltas, which we found in less time).

  Test                                    HEAD^                HEAD
  -------------------------------------------------------------------------------------
  5310.4: simulated fetch                 7.39(22.70+0.25)     5.64(11.43+0.22) -23.7%
  5310.8: clone (partial bitmap)          18.45(24.83+1.19)    19.94(28.40+1.36) +8.1%
  5311.31: server (128 days)              0.41(1.13+0.05)      0.34(0.72+0.02) -17.1%
  5311.32: size   (128 days)                         7.4M                 7.0M -4.8%
  5311.33: client (128 days)              1.33(1.49+0.06)      1.29(1.37+0.12) -3.0%

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Jeff King 2019-03-15 02:25:28 -04:00 committed by Junio C Hamano
parent 90ca149560
commit d4316604f8
5 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Documentation/config/pack.txt
View File

@ -124,6 +124,4 @@ pack.writeBitmapHashCache::
bitmapped and non-bitmapped objects (e.g., when serving a fetch bitmapped and non-bitmapped objects (e.g., when serving a fetch
between an older, bitmapped pack and objects that have been between an older, bitmapped pack and objects that have been
pushed since the last gc). The downside is that it consumes 4 pushed since the last gc). The downside is that it consumes 4
bytes per object of disk space, and that JGit's bitmap bytes per object of disk space. Defaults to true.
implementation does not understand it, causing it to complain if
Git and JGit are used on the same repository. Defaults to false.

2
builtin/pack-objects.c
View File

@ -97,7 +97,7 @@ static off_t reuse_packfile_offset;
static int use_bitmap_index_default = 1; static int use_bitmap_index_default = 1;
static int use_bitmap_index = -1; static int use_bitmap_index = -1;
static int write_bitmap_index; static int write_bitmap_index;
static uint16_t write_bitmap_options; static uint16_t write_bitmap_options = BITMAP_OPT_HASH_CACHE;
static int exclude_promisor_objects; static int exclude_promisor_objects;

3
t/perf/p5310-pack-bitmaps.sh
View File

@ -12,8 +12,7 @@ test_perf_large_repo
# We intentionally use the deprecated pack.writebitmaps # We intentionally use the deprecated pack.writebitmaps
# config so that we can test against older versions of git. # config so that we can test against older versions of git.
test_expect_success 'setup bitmap config' ' test_expect_success 'setup bitmap config' '
git config pack.writebitmaps true && git config pack.writebitmaps true
git config pack.writebitmaphashcache true
' '
test_perf 'repack to disk' ' test_perf 'repack to disk' '

1
t/perf/p5311-pack-bitmaps-fetch.sh
View File

@ -7,7 +7,6 @@ test_perf_default_repo
test_expect_success 'create bitmapped server repo' ' test_expect_success 'create bitmapped server repo' '
git config pack.writebitmaps true && git config pack.writebitmaps true &&
git config pack.writebitmaphashcache true &&
git repack -ad git repack -ad
' '

3
t/t5310-pack-bitmaps.sh
View File

@ -34,8 +34,7 @@ test_expect_success 'setup repo with moderate-sized history' '
bitmaptip=$(git rev-parse master) && bitmaptip=$(git rev-parse master) &&
blob=$(echo tagged-blob | git hash-object -w --stdin) && blob=$(echo tagged-blob | git hash-object -w --stdin) &&
git tag tagged-blob $blob && git tag tagged-blob $blob &&
git config repack.writebitmaps true && git config repack.writebitmaps true
git config pack.writebitmaphashcache true
' '
test_expect_success 'full repack creates bitmaps' ' test_expect_success 'full repack creates bitmaps' '