p5302: create the repo in each index-pack test
authorJeff King <peff@peff.net>
Mon, 22 Apr 2019 21:19:52 +0000 (17:19 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 23 Apr 2019 00:56:44 +0000 (09:56 +0900)
The p5302 script runs "index-pack --stdin" in each timing test. It does
two things to try to get good timings:

  1. we do the repo creation in a separate (non-timed) setup test, so
     that our timing is purely the index-pack run

  2. we use a separate repo for each test; this is important because the
     presence of existing objects in the repo influences the result
     (because we'll end up doing collision checks against them)

But this forgets one thing: we generally run each timed test multiple
times to reduce the impact of noise. Which means that repeats of each
test after the first will be subject to the collision slowdown from
point 2, and we'll generally just end up taking the first time anyway.

Instead, let's create the repo in the test (effectively undoing point
1). That does add a constant amount of extra work to each iteration, but
it's quite small compared to the actual effects we're interested in
measuring.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/perf/p5302-pack-index.sh

index 99bdb16..a9b3e11 100755 (executable)
@@ -13,35 +13,40 @@ test_expect_success 'repack' '
        export PACK
 '
 
-test_expect_success 'create target repositories' '
-       for repo in t1 t2 t3 t4 t5 t6
-       do
-               git init --bare $repo
-       done
-'
-
 test_perf 'index-pack 0 threads' '
-       GIT_DIR=t1 git index-pack --threads=1 --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git git index-pack --threads=1 --stdin < $PACK
 '
 
 test_perf 'index-pack 1 thread ' '
-       GIT_DIR=t2 GIT_FORCE_THREADS=1 git index-pack --threads=1 --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git GIT_FORCE_THREADS=1 git index-pack --threads=1 --stdin < $PACK
 '
 
 test_perf 'index-pack 2 threads' '
-       GIT_DIR=t3 git index-pack --threads=2 --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git git index-pack --threads=2 --stdin < $PACK
 '
 
 test_perf 'index-pack 4 threads' '
-       GIT_DIR=t4 git index-pack --threads=4 --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git git index-pack --threads=4 --stdin < $PACK
 '
 
 test_perf 'index-pack 8 threads' '
-       GIT_DIR=t5 git index-pack --threads=8 --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git git index-pack --threads=8 --stdin < $PACK
 '
 
 test_perf 'index-pack default number of threads' '
-       GIT_DIR=t6 git index-pack --stdin < $PACK
+       rm -rf repo.git &&
+       git init --bare repo.git &&
+       GIT_DIR=repo.git git index-pack --stdin < $PACK
 '
 
 test_done