Skip to content

Commit

Permalink
MINOR: [Go][CI][Benchmarking] Add timeout to go benchmarks (#14982)
Browse files Browse the repository at this point in the history
As mentioned by @ kou here: #14952 (comment) it looks like we crossed a threshold on the Go benchmarks and have added enough benchmarks for the Compute module that they are collectively taking more than 10m to run (which is the default timeout). So this updates the timeout used for running the benchmarks to 20 minutes.

CC @ austin3dickey 

Authored-by: Matt Topol <zotthewizard@gmail.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
  • Loading branch information
zeroshade committed Dec 15, 2022
1 parent caa6bf8 commit 2708c08
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion ci/scripts/go_bench.sh
Expand Up @@ -41,7 +41,10 @@ pushd ${source_dir}
go test -bench=. -benchmem -run=^$ ./... | tee bench_stat.dat

if verlte "1.18" "${ver#go}"; then
go test -bench=. -benchmem -run=^$ ./arrow/compute | tee bench_stat_compute.dat
# lots of benchmarks, they can take a while
# the timeout is for *ALL* benchmarks together,
# not per benchmark
go test -timeout 20m -bench=. -benchmem -run=^$ ./arrow/compute | tee bench_stat_compute.dat
fi

popd
Expand Down

0 comments on commit 2708c08

Please sign in to comment.