]> git.ipfire.org Git - thirdparty/git.git/commit - t/t3311-notes-merge-fanout.sh
t3311: use test_commit_bulk
authorJeff King <peff@peff.net>
Fri, 28 Jun 2019 09:41:35 +0000 (05:41 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 2 Jul 2019 17:11:54 +0000 (10:11 -0700)
commit737b19b50ccecfea56875472e3377ad989a1659f
treebbb9b85afafcf4755dfb5cb1aa304a3f8fa46411
parent1ac96cd1ba5938d4da251bf6447f11da8692db05
t3311: use test_commit_bulk

One of the tests in t3311 creates 300 commits by running "test_commit"
in a loop. This requires 900 processes. Instead, we can use
test_commit_bulk to do it with only four. This improves the runtime of
the script from:

  Benchmark #1: ./t3311-notes-merge-fanout.sh --root=/var/ram/git-tests
    Time (mean ± σ):      5.821 s ±  0.691 s    [User: 3.146 s, System: 2.782 s]
    Range (min … max):    4.783 s …  6.841 s    10 runs

to:

  Benchmark #1: ./t3311-notes-merge-fanout.sh --root=/var/ram/git-tests
    Time (mean ± σ):      1.743 s ±  0.116 s    [User: 1.144 s, System: 0.691 s]
    Range (min … max):    1.629 s …  1.994 s    10 runs

for an average speedup of over 70%.

Unfortunately we still have to run 300 instances of "git notes add",
since the point is to test the fanout that comes from adding notes one
by one.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t3311-notes-merge-fanout.sh