Skip to content

Commit 122739b

Browse files
committed
cmd/coordinator: update the timing hints table a bit
If the coordinator has a better guess as to how long various tests take, then it can do better critical path scheduling and reduce the overall time the sharded tests take to complete. This table still needs to die and be based on recent empirical data, but at least it's more accurate now, after a long delay in being updated. Update is from golang/go#17104 (comment) Updates golang/go#17104 Change-Id: I115aad23fbdb0cde1b196e71a4131fbe36480cc0 Reviewed-on: https://go-review.googlesource.com/29167 Reviewed-by: David Crawshaw <[email protected]>
1 parent 94d0141 commit 122739b

File tree

1 file changed

+40
-3
lines changed

1 file changed

+40
-3
lines changed

cmd/coordinator/coordinator.go

Lines changed: 40 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2091,9 +2091,46 @@ var fixedTestDuration = map[string]Seconds{
20912091
"test:4_5": 10,
20922092
"codewalk": 2.42,
20932093
"api": 7.38,
2094-
}
2095-
2096-
// testDuration predicts how long the dist test 'name' will take.
2094+
"go_test_bench:compress/bzip2": 3.059513602,
2095+
"go_test_bench:image/jpeg": 3.143345345,
2096+
"go_test_bench:encoding/hex": 3.182452293,
2097+
"go_test_bench:expvar": 3.490162906,
2098+
"go_test_bench:crypto/cipher": 3.609317114,
2099+
"go_test_bench:compress/lzw": 3.628982201,
2100+
"go_test_bench:database/sql": 3.693163398,
2101+
"go_test_bench:math/rand": 3.807438591,
2102+
"go_test_bench:bufio": 3.882166683,
2103+
"go_test_bench:context": 4.038173785,
2104+
"go_test_bench:hash/crc32": 4.107135055,
2105+
"go_test_bench:unicode/utf8": 4.205641826,
2106+
"go_test_bench:regexp/syntax": 4.587359311,
2107+
"go_test_bench:sort": 4.660599666,
2108+
"go_test_bench:math/cmplx": 5.311264213,
2109+
"go_test_bench:encoding/gob": 5.326788419,
2110+
"go_test_bench:reflect": 5.777081055,
2111+
"go_test_bench:image/png": 6.12439885,
2112+
"go_test_bench:html/template": 6.765132418,
2113+
"go_test_bench:fmt": 7.476528843,
2114+
"go_test_bench:sync": 7.526458261,
2115+
"go_test_bench:archive/zip": 7.782424696,
2116+
"go_test_bench:regexp": 8.428459563,
2117+
"go_test_bench:image/draw": 8.666510786,
2118+
"go_test_bench:strings": 10.836201759,
2119+
"go_test_bench:time": 10.952476479,
2120+
"go_test_bench:image/gif": 11.373276098,
2121+
"go_test_bench:encoding/json": 11.547950173,
2122+
"go_test_bench:crypto/tls": 11.548834754,
2123+
"go_test_bench:strconv": 12.819669296,
2124+
"go_test_bench:math": 13.7889302,
2125+
"go_test_bench:net": 14.845086695,
2126+
"go_test_bench:net/http": 15.288519219,
2127+
"go_test_bench:bytes": 15.809308703,
2128+
"go_test_bench:index/suffixarray": 23.69239388,
2129+
"go_test_bench:compress/flate": 26.906228664,
2130+
"go_test_bench:math/big": 28.82127674,
2131+
}
2132+
2133+
// testDuration predicts how long the dist test 'name' will take 'name' will take.
20972134
// It's only a scheduling guess.
20982135
func testDuration(name string) time.Duration {
20992136
if secs, ok := fixedTestDuration[name]; ok {

0 commit comments

Comments
 (0)