]> git.ipfire.org Git - thirdparty/git.git/commit
t/perf: drop p5311's lookup-table permutation
authorTaylor Blau <me@ttaylorr.com>
Tue, 2 Jun 2026 22:21:44 +0000 (18:21 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 2 Jun 2026 23:52:59 +0000 (08:52 +0900)
commit7aebfd0e08bb5469b2d38d04c651151dcda4319f
tree14135137992e3fe078ec88eed38fca52250b4c36
parent59cccb3b0c4aa385c41535cc11e6c22e80070633
t/perf: drop p5311's lookup-table permutation

p5311 measures the cost of serving a fetch from a bitmapped pack and
indexing the resulting pack on the client. Since 761416ef91d
(bitmap-lookup-table: add performance tests for lookup table,
2022-08-14), p5311 effectively runs itself twice: once with the bitmap's
lookup table extension enabled, and again with it disabled.

This comparison has served its useful purpose, as the lookup table is
almost four years old, and the de-facto default in server-side Git
deployments.

A following commit will want to test a different combination (repacking
with and without '--path-walk' instead of the lookup table). Instead of
multiplying the current test count by two again to produce four
variations of `test_fetch_bitmaps()`, drop the lookup table option to
reduce the number of perf tests we run. Retain `test_fetch_bitmaps()`
itself, since we will use this in the future for the new
parameterization.

(As an aside, a future commit outside of this series will adjust the
default value of 'pack.writeBitmapLookupTable' to "true", matching the
de-facto norm for deployments where the existence of bitmap lookup
tables is meaningful. Punt on that to a later series and instead make
the minimal change for now.)

Suggested-by: Derrick Stolee <stolee@gmail.com>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/perf/p5311-pack-bitmaps-fetch.sh