[PATCH v3 6/6] p5310-pack-bitmaps.sh: remove pack.writeReverseIndex
- From
Abhradeep Chakraborty via GitGitGadget <gitgitgadget@gmail.com>
- Date
- Jul 4, 2022, 08:46 UTC
- Message-ID
- <4f9f10494855265133bc315fccedf81ced65ce83.1656924376.git.gitgitgadget@gmail.com>
- In-Reply-To
- <pull.1266.v3.git.1656924376.gitgitgadget@gmail.com>
From: Abhradeep Chakraborty <chakrabortyabhradeep79@gmail.com>
The previous change enables the `pack.writereverseindex` to see the effect of writing reverse index in the performance test.
Remove the `pack.writeReverseIndex` configuration.
Below is the result of performance test. Output format is in seconds.
Test this tree
------------------------------------------------------------------------
5310.4: repack to disk (lookup=false) 293.80(251.30+14.30)
5310.5: simulated clone 12.50(5.15+1.36)
5310.6: simulated fetch 1.83(2.90+0.23)
5310.7: pack to file (bitmap) 39.70(20.25+7.14)
5310.8: rev-list (commits) 1.00(0.60+0.13)
5310.9: rev-list (objects) 4.11(4.00+0.10)
5310.10: rev-list with tag negated via --not 0.07(0.02+0.05)
--all (objects)
5310.11: rev-list with negative tag (objects) 0.23(0.16+0.06)
5310.12: rev-list count with blob:none 0.27(0.18+0.08)
5310.13: rev-list count with blob:limit=1k 6.41(5.98+0.41)
5310.14: rev-list count with tree:0 0.26(0.18+0.07)
5310.15: simulated partial clone 4.34(3.29+0.37)
5310.19: repack to disk (lookup=true) 250.93(171.97+20.78)
5310.20: simulated clone 10.80(5.14+1.06)
5310.21: simulated fetch 0.71(0.79+0.16)
5310.22: pack to file (bitmap) 39.49(20.19+6.98)
5310.23: rev-list (commits) 0.81(0.48+0.09)
5310.24: rev-list (objects) 3.48(3.38+0.09)
5310.25: rev-list with tag negated via --not 0.04(0.00+0.03)
--all (objects)
5310.26: rev-list with negative tag (objects) 0.22(0.16+0.05)
5310.27: rev-list count with blob:none 0.22(0.16+0.05)
5310.28: rev-list count with blob:limit=1k 6.21(5.76+0.29)
5310.29: rev-list count with tree:0 0.23(0.16+0.06)
5310.30: simulated partial clone 4.53(3.14+0.39)Tests 4-15 are without the use of lookup table. The rests are repeatation of the previous tests but using lookup table.
Mentored-by: Taylor Blau <me@ttaylorr.com> Co-Mentored-by: Kaartic Sivaraam <kaartic.sivaraam@gmail.com> Signed-off-by: Abhradeep Chakraborty <chakrabortyabhradeep79@gmail.com> --- t/perf/p5310-pack-bitmaps.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/t/perf/p5310-pack-bitmaps.sh b/t/perf/p5310-pack-bitmaps.sh index 1ad3c3f14c6..ac5b7341e8e 100755 --- a/t/perf/p5310-pack-bitmaps.sh +++ b/t/perf/p5310-pack-bitmaps.sh @@ -13,8 +13,7 @@ test_perf_large_repo # We intentionally use the deprecated pack.writebitmaps # config so that we can test against older versions of git. test_expect_success 'setup bitmap config' ' - git config pack.writebitmaps true && - git config pack.writeReverseIndex true + git config pack.writebitmaps true ' test_bitmap () {
-- gitgitgadget