]> Cypherpunks repositories - gostls13.git/commit
cmd/pack: buffer writes in TestLargeDefs
authorAnthony Martin <ality@pbrane.org>
Fri, 16 May 2014 03:12:06 +0000 (20:12 -0700)
committerRob Pike <r@golang.org>
Fri, 16 May 2014 03:12:06 +0000 (20:12 -0700)
commitc6aa2e5ac8097f9491a407c3bb2385159d9aed32
tree7fb5941023f324f10fcfde4a5a25a989112c92e3
parent74fe67f22e41f06ab93fc5123964f144040191aa
cmd/pack: buffer writes in TestLargeDefs

TestLargeDefs was issuing over one million small writes to
create a 7MB file (large.go). This is quite slow on Plan 9
since our disk file systems aren't very fast and they're
usually accessed over the network.

Buffering the writes makes the test about six times faster.
Even on Linux, it's about 1.5 times faster.

Here are the results on a slow Plan 9 machine:

Before:
        % ./pack.test -test.v -test.run TestLargeDefs
        === RUN TestLargeDefs
        --- PASS: TestLargeDefs (125.11 seconds)
        PASS

After:
        % ./pack.test -test.v -test.run TestLargeDefs
        === RUN TestLargeDefs
        --- PASS: TestLargeDefs (20.835 seconds)
        PASS

LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/95040044
src/cmd/pack/pack_test.go