tree c169dce7aff7d216b3eb3ddc8f76c7e89d10ec01
parent df0ca34f7b66ed1f05972f6d22b4c2f4c8db0b4c
author Bryan C. Mills <bcmills@google.com> 1639168826 -0500
committer Bryan Mills <bcmills@google.com> 1639170431 +0000

sweet/benchmarks/internal/pool: use errgroup to manage errors and cancellation

Also simplify concurrency in TestPoolCancel: t.Fatalf was called in a
background goroutine, which is not allowed (because it invokes
runtime.Goexit), but the background goroutine isn't really necessary
anyway. (If the test deadlocks, it's more useful to just let it time
out anyway so that the test will give us a useful goroutine dump.)

Fixes golang/go#50096

Change-Id: Ica9abf427f662f4ed1f7cd64ef906b49744009ef
Reviewed-on: https://go-review.googlesource.com/c/benchmarks/+/370974
Trust: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>