From: Jeff King <peff@peff.net> To: Taylor Blau <me@ttaylorr.com> Cc: git@vger.kernel.org, dstolee@microsoft.com, gitster@pobox.com, jonathantanmy@google.com Subject: Re: [PATCH v3 18/25] t5326: test multi-pack bitmap behavior Date: Thu, 12 Aug 2021 17:02:26 -0400 [thread overview] Message-ID: <YRWMYg2rvv7HjGE+@coredump.intra.peff.net> (raw) In-Reply-To: <3258ccfc1cc99038e43a37bd2d53c9d30a4f22ae.1627420428.git.me@ttaylorr.com> On Tue, Jul 27, 2021 at 05:20:10PM -0400, Taylor Blau wrote: > diff --git a/t/t5326-multi-pack-bitmaps.sh b/t/t5326-multi-pack-bitmaps.sh > new file mode 100755 > index 0000000000..c1b7d633e2 > --- /dev/null > +++ b/t/t5326-multi-pack-bitmaps.sh > @@ -0,0 +1,277 @@ > +#!/bin/sh > + > +test_description='exercise basic multi-pack bitmap functionality' > +. ./test-lib.sh > +. "${TEST_DIRECTORY}/lib-bitmap.sh" > + > +# We'll be writing our own midx and bitmaps, so avoid getting confused by the > +# automatic ones. > +GIT_TEST_MULTI_PACK_INDEX=0 > +GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP=0 This latter variable doesn't do anything at this point in the series. Probably not a big deal (it is simply a noop until then), but if it's not hard, it may make sense to bump the "respect ... WRITE_BITMAP" patch earlier in the series. > +test_expect_success 'create single-pack midx with bitmaps' ' > + git repack -ad && > + git multi-pack-index write --bitmap && > + test_path_is_file $midx && > + test_path_is_file $midx-$(midx_checksum $objdir).bitmap > +' > + > +basic_bitmap_tests We can't use a midx bitmap without a .rev file. The basic_bitmap_tests function covers that, but I wonder if we should also check: test_path_is_file $midx-$(midx_checksum $objdir).rev in that first test. > +test_expect_success 'create new additional packs' ' > + for i in $(test_seq 1 16) > + do > + test_commit "$i" && > + git repack -d > + done && This loop needs an "|| return 1" inside to catch &&-chain problems (not that we expect "repack -d" to fail, but just on principle). > + git checkout -b other2 HEAD~8 && > + for i in $(test_seq 1 8) > + do > + test_commit "side-$i" && > + git repack -d > + done && Ditto here. > +test_expect_success 'create multi-pack midx with bitmaps' ' > + git multi-pack-index write --bitmap && > + > + ls $objdir/pack/pack-*.pack >packs && > + test_line_count = 25 packs && > + > + test_path_is_file $midx && > + test_path_is_file $midx-$(midx_checksum $objdir).bitmap > +' Possible spot for checking the .rev file again (though really, it is belt-and-suspenders at this point). > +basic_bitmap_tests I love how the earlier refactoring made it easy to test the single- and multi-pack cases thoroughly. > +test_expect_success '--no-bitmap is respected when bitmaps exist' ' > + git multi-pack-index write --bitmap && > + > + test_commit respect--no-bitmap && > + GIT_TEST_MULTI_PACK_INDEX=0 git repack -d && Do we need to set this env variable? We've already set it to 0 at the top of the script. > + test_path_is_file $midx && > + test_path_is_file $midx-$(midx_checksum $objdir).bitmap && > + > + git multi-pack-index write --no-bitmap && > + > + test_path_is_file $midx && > + test_path_is_missing $midx-$(midx_checksum $objdir).bitmap > +' OK, so we expect "--no-bitmap" to drop the bitmap (just like it does for a regular pack bitmap). Makes sense. We probably should check: test_path_is_missing $midx-$(midx_checksum $objdir).rev here, too (unlike the other spots, it isn't redundant; we could leave a stale file around and likely nobody would notice). > +test_expect_success 'setup midx with base from later pack' ' > + # Write a and b so that "a" is a delta on top of base "b", since Git > + # prefers to delete contents out of a base rather than add to a shorter > + # object. > + test_seq 1 128 >a && > + test_seq 1 130 >b && > + > + git add a b && > + git commit -m "initial commit" && > + > + a=$(git rev-parse HEAD:a) && > + b=$(git rev-parse HEAD:b) && > + > + # In the first pack, "a" is stored as a delta to "b". > + p1=$(git pack-objects .git/objects/pack/pack <<-EOF > + $a > + $b > + EOF > + ) && This is brittle with respect to Git's delta heuristics, of course, but I don't think there's a better way to do it with pack-objects. And this is not the first test to make similar assumptions. I think you can construct a known set of deltas using lib-pack.sh. It may get a bit complicated. As an alternative, maybe it makes sense to confirm that the deltas are set up as expected? You can do it with cat-file --batch-check. > +test_expect_success 'removing a MIDX clears stale bitmaps' ' > + rm -fr repo && > + git init repo && > + test_when_finished "rm -fr repo" && > + ( > + cd repo && > + test_commit base && > + git repack && > + git multi-pack-index write --bitmap && > + > + # Write a MIDX and bitmap; remove the MIDX but leave the bitmap. > + stale_bitmap=$midx-$(midx_checksum $objdir).bitmap && > + rm $midx && > + > + # Then write a new MIDX. > + test_commit new && > + git repack && > + git multi-pack-index write --bitmap && > + > + test_path_is_file $midx && > + test_path_is_file $midx-$(midx_checksum $objdir).bitmap && > + test_path_is_missing $stale_bitmap > + ) Another spot where we might want to check that the stale .rev file has gone away (and optionally that the new one was written; I haven't noted all of those, though). > +test_expect_success 'pack.preferBitmapTips' ' > + git init repo && > + test_when_finished "rm -fr repo" && > + ( > + cd repo && > + > + test_commit_bulk --message="%s" 103 && > + > + git log --format="%H" >commits.raw && > + sort <commits.raw >commits && > + > + git log --format="create refs/tags/%s %H" HEAD >refs && > + git update-ref --stdin <refs && > + > + git multi-pack-index write --bitmap && > + test_path_is_file $midx && > + test_path_is_file $midx-$(midx_checksum $objdir).bitmap && > + > + test-tool bitmap list-commits | sort >bitmaps && > + comm -13 bitmaps commits >before && > + test_line_count = 1 before && > + > + perl -ne "printf(\"create refs/tags/include/%d \", $.); print" \ > + <before | git update-ref --stdin && > + > + rm -fr $midx-$(midx_checksum $objdir).bitmap && > + rm -fr $midx-$(midx_checksum $objdir).rev && > + rm -fr $midx && > + > + git -c pack.preferBitmapTips=refs/tags/include \ > + multi-pack-index write --bitmap && > + test-tool bitmap list-commits | sort >bitmaps && > + comm -13 bitmaps commits >after && > + > + ! test_cmp before after > + ) > +' OK, so we are not depending on any _specific_ commits to get bitmapped, but just confirming that we have some impact. That may be the best we can do given that we are subject to the bitmap code's heuristics (and anyway, this is exactly what the pack version does). Any other parts of the patch that I didn't quote looked very good to me. I'm happy to have such a thorough set of tests. -Peff
next prev parent reply other threads:[~2021-08-12 21:02 UTC|newest] Thread overview: 273+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-09 18:10 [PATCH 00/22] multi-pack reachability bitmaps Taylor Blau 2021-04-09 18:10 ` [PATCH 01/22] pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps Taylor Blau 2021-04-09 18:10 ` [PATCH 02/22] pack-bitmap-write.c: gracefully fail to write non-closed bitmaps Taylor Blau 2021-04-16 2:46 ` Jonathan Tan 2021-04-09 18:10 ` [PATCH 03/22] pack-bitmap-write.c: free existing bitmaps Taylor Blau 2021-04-09 18:10 ` [PATCH 04/22] Documentation: build 'technical/bitmap-format' by default Taylor Blau 2021-04-09 18:11 ` [PATCH 05/22] Documentation: describe MIDX-based bitmaps Taylor Blau 2021-04-09 18:11 ` [PATCH 06/22] midx: make a number of functions non-static Taylor Blau 2021-04-09 18:11 ` [PATCH 07/22] midx: clear auxiliary .rev after replacing the MIDX Taylor Blau 2021-04-09 18:11 ` [PATCH 08/22] midx: respect 'core.multiPackIndex' when writing Taylor Blau 2021-04-09 18:11 ` [PATCH 09/22] pack-bitmap.c: introduce 'bitmap_num_objects()' Taylor Blau 2021-04-09 18:11 ` [PATCH 10/22] pack-bitmap.c: introduce 'nth_bitmap_object_oid()' Taylor Blau 2021-04-09 18:11 ` [PATCH 11/22] pack-bitmap.c: introduce 'bitmap_is_preferred_refname()' Taylor Blau 2021-04-09 18:11 ` [PATCH 12/22] pack-bitmap: read multi-pack bitmaps Taylor Blau 2021-04-16 2:39 ` Jonathan Tan 2021-04-16 3:13 ` Taylor Blau 2021-04-09 18:11 ` [PATCH 13/22] pack-bitmap: write " Taylor Blau 2021-05-04 5:02 ` Jonathan Tan 2021-05-06 20:18 ` Taylor Blau 2021-05-06 22:00 ` Jonathan Tan 2021-04-09 18:11 ` [PATCH 14/22] t5310: move some tests to lib-bitmap.sh Taylor Blau 2021-04-09 18:11 ` [PATCH 15/22] t/helper/test-read-midx.c: add --checksum mode Taylor Blau 2021-04-09 18:12 ` [PATCH 16/22] t5326: test multi-pack bitmap behavior Taylor Blau 2021-05-04 17:51 ` Jonathan Tan 2021-04-09 18:12 ` [PATCH 17/22] t5310: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP Taylor Blau 2021-04-09 18:12 ` [PATCH 18/22] t5319: don't write MIDX bitmaps in t5319 Taylor Blau 2021-04-09 18:12 ` [PATCH 19/22] t7700: update to work with MIDX bitmap test knob Taylor Blau 2021-04-09 18:12 ` [PATCH 20/22] midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP' Taylor Blau 2021-04-09 18:12 ` [PATCH 21/22] p5310: extract full and partial bitmap tests Taylor Blau 2021-04-09 18:12 ` [PATCH 22/22] p5326: perf tests for MIDX bitmaps Taylor Blau 2021-05-04 18:00 ` Jonathan Tan 2021-05-05 0:55 ` Junio C Hamano 2021-06-21 22:24 ` [PATCH v2 00/24] multi-pack reachability bitmaps Taylor Blau 2021-06-21 22:24 ` [PATCH v2 01/24] pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps Taylor Blau 2021-06-24 23:02 ` Ævar Arnfjörð Bjarmason 2021-07-14 17:24 ` Taylor Blau 2021-07-21 9:45 ` Jeff King 2021-07-21 17:15 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 02/24] pack-bitmap-write.c: gracefully fail to write non-closed bitmaps Taylor Blau 2021-06-24 23:23 ` Ævar Arnfjörð Bjarmason 2021-07-14 17:32 ` Taylor Blau 2021-07-14 18:44 ` Ævar Arnfjörð Bjarmason 2021-07-21 9:53 ` Jeff King 2021-07-21 9:50 ` Jeff King 2021-07-21 17:20 ` Taylor Blau 2021-07-23 7:37 ` Jeff King 2021-07-26 18:48 ` Taylor Blau 2021-07-27 17:11 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 03/24] pack-bitmap-write.c: free existing bitmaps Taylor Blau 2021-07-21 9:54 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 04/24] Documentation: build 'technical/bitmap-format' by default Taylor Blau 2021-06-24 23:35 ` Ævar Arnfjörð Bjarmason 2021-07-14 17:41 ` Taylor Blau 2021-07-14 22:58 ` Ævar Arnfjörð Bjarmason 2021-07-21 10:04 ` Jeff King 2021-07-21 10:10 ` Jeff King 2021-07-21 9:58 ` Jeff King 2021-07-21 10:08 ` Jeff King 2021-07-21 17:23 ` Taylor Blau 2021-07-23 7:39 ` Jeff King 2021-07-26 18:49 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 05/24] Documentation: describe MIDX-based bitmaps Taylor Blau 2021-07-21 10:18 ` Jeff King 2021-07-21 17:53 ` Taylor Blau 2021-07-23 7:45 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 06/24] midx: make a number of functions non-static Taylor Blau 2021-06-24 23:42 ` Ævar Arnfjörð Bjarmason 2021-07-14 23:01 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 07/24] midx: clear auxiliary .rev after replacing the MIDX Taylor Blau 2021-07-21 10:19 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 08/24] midx: respect 'core.multiPackIndex' when writing Taylor Blau 2021-06-24 23:43 ` Ævar Arnfjörð Bjarmason 2021-07-21 10:23 ` Jeff King 2021-07-21 19:22 ` Taylor Blau 2021-07-23 8:29 ` Jeff King 2021-07-26 18:59 ` Taylor Blau 2021-07-26 22:14 ` Taylor Blau 2021-07-27 17:29 ` Jeff King 2021-07-27 17:36 ` Taylor Blau 2021-07-27 17:42 ` Jeff King 2021-07-27 17:47 ` Taylor Blau 2021-07-27 17:55 ` Jeff King 2021-07-27 20:05 ` Taylor Blau 2021-07-28 17:46 ` Jeff King 2021-07-29 19:44 ` Taylor Blau 2021-08-12 19:59 ` Jeff King 2021-07-27 17:17 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 09/24] midx: infer preferred pack when not given one Taylor Blau 2021-07-21 10:34 ` Jeff King 2021-07-21 20:16 ` Taylor Blau 2021-07-23 8:50 ` Jeff King 2021-07-26 19:44 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 10/24] pack-bitmap.c: introduce 'bitmap_num_objects()' Taylor Blau 2021-07-21 10:35 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 11/24] pack-bitmap.c: introduce 'nth_bitmap_object_oid()' Taylor Blau 2021-06-24 14:59 ` Taylor Blau 2021-07-21 10:37 ` Jeff King 2021-07-21 10:38 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 12/24] pack-bitmap.c: introduce 'bitmap_is_preferred_refname()' Taylor Blau 2021-07-21 10:39 ` Jeff King 2021-07-21 20:18 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 13/24] pack-bitmap: read multi-pack bitmaps Taylor Blau 2021-07-21 11:32 ` Jeff King 2021-07-21 23:01 ` Taylor Blau 2021-07-23 9:40 ` Jeff King 2021-07-23 10:00 ` Jeff King 2021-07-26 20:36 ` Taylor Blau 2021-06-21 22:25 ` [PATCH v2 14/24] pack-bitmap: write " Taylor Blau 2021-06-24 23:45 ` Ævar Arnfjörð Bjarmason 2021-07-15 14:33 ` Taylor Blau 2021-07-21 12:09 ` Jeff King 2021-07-26 18:12 ` Taylor Blau 2021-07-26 18:23 ` Taylor Blau 2021-07-27 17:11 ` Jeff King 2021-07-27 20:33 ` Taylor Blau 2021-07-28 17:52 ` Jeff King 2021-07-29 19:33 ` Taylor Blau 2021-08-12 20:00 ` Jeff King 2021-06-21 22:25 ` [PATCH v2 15/24] t5310: move some tests to lib-bitmap.sh Taylor Blau 2021-06-21 22:25 ` [PATCH v2 16/24] t/helper/test-read-midx.c: add --checksum mode Taylor Blau 2021-06-21 22:25 ` [PATCH v2 17/24] t5326: test multi-pack bitmap behavior Taylor Blau 2021-06-21 22:25 ` [PATCH v2 18/24] t0410: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP Taylor Blau 2021-06-21 22:25 ` [PATCH v2 19/24] t5310: " Taylor Blau 2021-06-21 22:25 ` [PATCH v2 20/24] t5319: don't write MIDX bitmaps in t5319 Taylor Blau 2021-06-21 22:25 ` [PATCH v2 21/24] t7700: update to work with MIDX bitmap test knob Taylor Blau 2021-06-21 22:25 ` [PATCH v2 22/24] midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP' Taylor Blau 2021-06-25 0:03 ` Ævar Arnfjörð Bjarmason 2021-06-21 22:25 ` [PATCH v2 23/24] p5310: extract full and partial bitmap tests Taylor Blau 2021-06-21 22:26 ` [PATCH v2 24/24] p5326: perf tests for MIDX bitmaps Taylor Blau 2021-06-25 9:06 ` [PATCH v2 00/24] multi-pack reachability bitmaps Ævar Arnfjörð Bjarmason 2021-07-15 14:36 ` Taylor Blau 2021-07-21 12:12 ` Jeff King 2021-07-27 21:19 ` [PATCH v3 00/25] " Taylor Blau 2021-07-27 21:19 ` [PATCH v3 01/25] pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps Taylor Blau 2021-07-27 21:19 ` [PATCH v3 02/25] pack-bitmap-write.c: gracefully fail to write non-closed bitmaps Taylor Blau 2021-07-27 21:19 ` [PATCH v3 03/25] pack-bitmap-write.c: free existing bitmaps Taylor Blau 2021-07-27 21:19 ` [PATCH v3 04/25] Documentation: describe MIDX-based bitmaps Taylor Blau 2021-07-27 21:19 ` [PATCH v3 05/25] midx: clear auxiliary .rev after replacing the MIDX Taylor Blau 2021-07-27 21:19 ` [PATCH v3 06/25] midx: reject empty `--preferred-pack`'s Taylor Blau 2021-07-27 21:19 ` [PATCH v3 07/25] midx: infer preferred pack when not given one Taylor Blau 2021-07-27 21:19 ` [PATCH v3 08/25] midx: close linked MIDXs, avoid leaking memory Taylor Blau 2021-07-27 21:19 ` [PATCH v3 09/25] midx: avoid opening multiple MIDXs when writing Taylor Blau 2021-07-29 19:30 ` Taylor Blau 2021-08-12 20:15 ` Jeff King 2021-08-12 20:22 ` Jeff King 2021-08-12 21:20 ` Taylor Blau 2021-07-27 21:19 ` [PATCH v3 10/25] pack-bitmap.c: introduce 'bitmap_num_objects()' Taylor Blau 2021-07-27 21:19 ` [PATCH v3 11/25] pack-bitmap.c: introduce 'nth_bitmap_object_oid()' Taylor Blau 2021-07-27 21:19 ` [PATCH v3 12/25] pack-bitmap.c: introduce 'bitmap_is_preferred_refname()' Taylor Blau 2021-07-27 21:19 ` [PATCH v3 13/25] pack-bitmap.c: avoid redundant calls to try_partial_reuse Taylor Blau 2021-07-27 21:19 ` [PATCH v3 14/25] pack-bitmap: read multi-pack bitmaps Taylor Blau 2021-07-27 21:20 ` [PATCH v3 15/25] pack-bitmap: write " Taylor Blau 2021-07-27 21:20 ` [PATCH v3 16/25] t5310: move some tests to lib-bitmap.sh Taylor Blau 2021-08-12 20:25 ` Jeff King 2021-07-27 21:20 ` [PATCH v3 17/25] t/helper/test-read-midx.c: add --checksum mode Taylor Blau 2021-08-12 20:31 ` Jeff King 2021-08-12 21:31 ` Taylor Blau 2021-07-27 21:20 ` [PATCH v3 18/25] t5326: test multi-pack bitmap behavior Taylor Blau 2021-08-12 21:02 ` Jeff King [this message] 2021-08-12 21:07 ` Jeff King 2021-08-12 22:38 ` Taylor Blau 2021-08-12 23:23 ` Jeff King 2021-07-27 21:20 ` [PATCH v3 19/25] t0410: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP Taylor Blau 2021-07-27 21:20 ` [PATCH v3 20/25] t5310: " Taylor Blau 2021-07-27 21:20 ` [PATCH v3 21/25] t5319: don't write MIDX bitmaps in t5319 Taylor Blau 2021-07-27 21:20 ` [PATCH v3 22/25] t7700: update to work with MIDX bitmap test knob Taylor Blau 2021-07-27 21:20 ` [PATCH v3 23/25] midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP' Taylor Blau 2021-08-12 21:09 ` Jeff King 2021-07-27 21:20 ` [PATCH v3 24/25] p5310: extract full and partial bitmap tests Taylor Blau 2021-07-27 21:20 ` [PATCH v3 25/25] p5326: perf tests for MIDX bitmaps Taylor Blau 2021-08-12 21:18 ` Jeff King 2021-08-12 21:21 ` [PATCH v3 00/25] multi-pack reachability bitmaps Jeff King 2021-08-12 22:41 ` Taylor Blau 2021-08-24 16:15 ` [PATCH v4 " Taylor Blau 2021-08-24 16:15 ` [PATCH v4 01/25] pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps Taylor Blau 2021-08-24 16:15 ` [PATCH v4 02/25] pack-bitmap-write.c: gracefully fail to write non-closed bitmaps Taylor Blau 2021-08-24 16:15 ` [PATCH v4 03/25] pack-bitmap-write.c: free existing bitmaps Taylor Blau 2021-08-24 16:15 ` [PATCH v4 04/25] Documentation: describe MIDX-based bitmaps Taylor Blau 2021-08-24 16:16 ` [PATCH v4 05/25] midx: clear auxiliary .rev after replacing the MIDX Taylor Blau 2021-08-24 20:27 ` Junio C Hamano 2021-08-24 20:34 ` Taylor Blau 2021-08-24 21:12 ` Junio C Hamano 2021-08-24 21:24 ` Taylor Blau 2021-08-24 22:01 ` Taylor Blau 2021-08-24 22:04 ` Junio C Hamano 2021-08-24 22:06 ` Junio C Hamano 2021-08-24 22:10 ` Taylor Blau 2021-08-27 6:01 ` Junio C Hamano 2021-08-27 18:03 ` Taylor Blau 2021-08-29 22:56 ` Junio C Hamano 2021-08-30 0:07 ` Taylor Blau 2021-08-30 0:34 ` Junio C Hamano 2021-08-30 0:43 ` Taylor Blau 2021-08-30 22:10 ` brian m. carlson 2021-08-30 22:28 ` Junio C Hamano 2021-08-30 22:33 ` Taylor Blau 2021-08-31 5:19 ` Jeff King 2021-08-31 16:29 ` Junio C Hamano 2021-08-31 16:39 ` Taylor Blau 2021-08-31 17:44 ` Junio C Hamano 2021-08-31 18:48 ` Taylor Blau 2021-08-31 1:21 ` Derrick Stolee 2021-08-31 5:37 ` Jeff King 2021-08-31 16:33 ` Junio C Hamano 2021-08-31 16:43 ` Taylor Blau 2021-08-31 17:17 ` Derrick Stolee 2021-09-01 10:03 ` Jeff King 2021-08-24 16:16 ` [PATCH v4 06/25] midx: reject empty `--preferred-pack`'s Taylor Blau 2021-08-24 16:16 ` [PATCH v4 07/25] midx: infer preferred pack when not given one Taylor Blau 2021-08-24 16:16 ` [PATCH v4 08/25] midx: close linked MIDXs, avoid leaking memory Taylor Blau 2021-08-24 16:16 ` [PATCH v4 09/25] midx: avoid opening multiple MIDXs when writing Taylor Blau 2021-08-24 16:16 ` [PATCH v4 10/25] pack-bitmap.c: introduce 'bitmap_num_objects()' Taylor Blau 2021-08-24 16:16 ` [PATCH v4 11/25] pack-bitmap.c: introduce 'nth_bitmap_object_oid()' Taylor Blau 2021-08-24 16:16 ` [PATCH v4 12/25] pack-bitmap.c: introduce 'bitmap_is_preferred_refname()' Taylor Blau 2021-08-24 16:16 ` [PATCH v4 13/25] pack-bitmap.c: avoid redundant calls to try_partial_reuse Taylor Blau 2021-08-24 16:16 ` [PATCH v4 14/25] pack-bitmap: read multi-pack bitmaps Taylor Blau 2021-08-24 16:16 ` [PATCH v4 15/25] pack-bitmap: write " Taylor Blau 2021-08-24 16:16 ` [PATCH v4 16/25] t5310: move some tests to lib-bitmap.sh Taylor Blau 2021-08-24 16:16 ` [PATCH v4 17/25] t/helper/test-read-midx.c: add --checksum mode Taylor Blau 2021-08-24 16:16 ` [PATCH v4 18/25] t5326: test multi-pack bitmap behavior Taylor Blau 2021-08-24 16:16 ` [PATCH v4 19/25] t0410: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP Taylor Blau 2021-08-24 16:16 ` [PATCH v4 20/25] t5310: " Taylor Blau 2021-08-24 16:16 ` [PATCH v4 21/25] t5319: don't write MIDX bitmaps in t5319 Taylor Blau 2021-08-24 16:16 ` [PATCH v4 22/25] t7700: update to work with MIDX bitmap test knob Taylor Blau 2021-08-24 16:16 ` [PATCH v4 23/25] midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP' Taylor Blau 2021-08-24 16:16 ` [PATCH v4 24/25] p5310: extract full and partial bitmap tests Taylor Blau 2021-08-24 16:16 ` [PATCH v4 25/25] p5326: perf tests for MIDX bitmaps Taylor Blau 2021-08-25 0:28 ` [PATCH v4 00/25] multi-pack reachability bitmaps Jeff King 2021-08-25 2:10 ` Taylor Blau 2021-08-25 2:13 ` Taylor Blau 2021-08-25 7:36 ` Jeff King 2021-08-25 7:48 ` Johannes Berg 2021-08-26 18:49 ` Taylor Blau 2021-08-26 21:22 ` Taylor Blau 2021-08-27 21:30 ` Jeff King 2021-08-29 22:42 ` Junio C Hamano 2021-08-31 20:51 ` [PATCH v5 00/27] " Taylor Blau 2021-08-31 20:51 ` [PATCH v5 01/27] pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps Taylor Blau 2021-08-31 20:51 ` [PATCH v5 02/27] pack-bitmap-write.c: gracefully fail to write non-closed bitmaps Taylor Blau 2021-08-31 20:51 ` [PATCH v5 03/27] pack-bitmap-write.c: free existing bitmaps Taylor Blau 2021-08-31 20:51 ` [PATCH v5 04/27] Documentation: describe MIDX-based bitmaps Taylor Blau 2021-08-31 20:51 ` [PATCH v5 05/27] midx: disallow running outside of a repository Taylor Blau 2021-08-31 20:51 ` [PATCH v5 06/27] midx: fix `*.rev` cleanups with `--object-dir` Taylor Blau 2021-08-31 20:51 ` [PATCH v5 07/27] midx: clear auxiliary .rev after replacing the MIDX Taylor Blau 2021-08-31 20:52 ` [PATCH v5 08/27] midx: reject empty `--preferred-pack`'s Taylor Blau 2021-08-31 20:52 ` [PATCH v5 09/27] midx: infer preferred pack when not given one Taylor Blau 2021-08-31 20:52 ` [PATCH v5 10/27] midx: close linked MIDXs, avoid leaking memory Taylor Blau 2021-08-31 20:52 ` [PATCH v5 11/27] midx: avoid opening multiple MIDXs when writing Taylor Blau 2021-08-31 20:52 ` [PATCH v5 12/27] pack-bitmap.c: introduce 'bitmap_num_objects()' Taylor Blau 2021-08-31 20:52 ` [PATCH v5 13/27] pack-bitmap.c: introduce 'nth_bitmap_object_oid()' Taylor Blau 2021-08-31 20:52 ` [PATCH v5 14/27] pack-bitmap.c: introduce 'bitmap_is_preferred_refname()' Taylor Blau 2021-08-31 20:52 ` [PATCH v5 15/27] pack-bitmap.c: avoid redundant calls to try_partial_reuse Taylor Blau 2021-08-31 20:52 ` [PATCH v5 16/27] pack-bitmap: read multi-pack bitmaps Taylor Blau 2021-08-31 20:52 ` [PATCH v5 17/27] pack-bitmap: write " Taylor Blau 2021-08-31 20:52 ` [PATCH v5 18/27] t5310: move some tests to lib-bitmap.sh Taylor Blau 2021-08-31 20:52 ` [PATCH v5 19/27] t/helper/test-read-midx.c: add --checksum mode Taylor Blau 2021-08-31 20:52 ` [PATCH v5 20/27] t5326: test multi-pack bitmap behavior Taylor Blau 2021-08-31 20:52 ` [PATCH v5 21/27] t0410: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP Taylor Blau 2021-08-31 20:52 ` [PATCH v5 22/27] t5310: " Taylor Blau 2021-08-31 20:52 ` [PATCH v5 23/27] t5319: don't write MIDX bitmaps in t5319 Taylor Blau 2021-08-31 20:52 ` [PATCH v5 24/27] t7700: update to work with MIDX bitmap test knob Taylor Blau 2021-08-31 20:52 ` [PATCH v5 25/27] midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP' Taylor Blau 2021-08-31 20:52 ` [PATCH v5 26/27] p5310: extract full and partial bitmap tests Taylor Blau 2021-08-31 20:52 ` [PATCH v5 27/27] p5326: perf tests for MIDX bitmaps Taylor Blau 2021-09-01 18:07 ` [PATCH v5 00/27] multi-pack reachability bitmaps Junio C Hamano 2021-09-01 19:08 ` Taylor Blau 2021-09-01 19:23 ` Junio C Hamano 2021-09-01 20:34 ` Taylor Blau 2021-09-01 20:49 ` Junio C Hamano 2021-09-01 20:54 ` Taylor Blau 2021-09-02 9:40 ` Jeff King 2021-09-02 9:38 ` Jeff King 2021-09-02 9:45 ` Jeff King
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 List information: http://vger.kernel.org/majordomo-info.html * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YRWMYg2rvv7HjGE+@coredump.intra.peff.net \ --to=peff@peff.net \ --cc=dstolee@microsoft.com \ --cc=git@vger.kernel.org \ --cc=gitster@pobox.com \ --cc=jonathantanmy@google.com \ --cc=me@ttaylorr.com \ --subject='Re: [PATCH v3 18/25] t5326: test multi-pack bitmap behavior' \ /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
Code repositories for project(s) associated with this inbox: https://80x24.org/mirrors/git.git 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 read-only IMAP folder(s) and NNTP newsgroup(s).