tree 61eb81eec7d10083ed5c1d50ec5ddef6a8f7d8f7
parent 54da6ec3604bde35a9997f50f0f44e14da3c894f
author Kevin Lubick <kjlubick@google.com> 1613742653 -0500
committer Kevin Lubick <kjlubick@google.com> 1613996474 +0000

[gold] Make diffcalculators fully respect ctx.Done()

The big idea here is to replace a for loop over the
range of a channel with an infinite for loop that
has a select statement at the top to make use of
the ctx.

Reminder that errgroups do not actually kill all the
spawned goroutines for you - they must terminate
themselves by listening to the returned errctx.

Additionally, we close the channels if we can't calculate
the work. This assuredly stops the goroutines we stopped.

Bug: skia:10582
Change-Id: I21ebd22116127bce4ff39887c49cfa6d7462ecfc
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/372389
Reviewed-by: Leandro Lovisolo <lovisolo@google.com>