📚 背景知識(読んでから問題へ)
Worker Poolは「固定数のgoroutineでジョブを処理し、無制限にgoroutineを増やさない」ための並行処理パターンです。Phase 2で学んだgoroutine・channel・WaitGroupの基本要素を、実務で最も頻繁に登場する形に組み合わせたものだと言えます。
典型的な構成要素は3つです。
jobs := make(chan Job, N) // 仕事を流し込むchannel
results := make(chan Result, N) // 結果を受け取るchannel
var wg sync.WaitGroup // 全workerの完了を待つ
- なぜgoroutineを無制限に増やさないのか: 1リクエストごとに
go handle(req)していると、リクエストが殺到した瞬間にgoroutine数が青天井に増え、メモリとスケジューラの両方を圧迫します。Worker Poolは「同時に処理するジョブ数の上限」をnumWorkersという1つの数値でコード上に明示する設計です。 - channelの所有権(ownership): Goのchannelには「誰が閉じるか」という暗黙のルールがあります。送信する側(sender)だけがcloseしてよく、受信する側(receiver)は絶対にcloseしないという原則です。複数のgoroutineが同じchannelに書き込んでいる場合、closeしてよいのは「全員が書き込み終わったことを検知できる立場」の1箇所だけです。今日の問題は、このownershipの原則が破られたときに何が起きるかを、実際のバグとして体験します。
Worker Poolの正しい終了シーケンスは次の順序です。
- ジョブ送信側が全ジョブを送り終えたら
jobsをcloseする - 各workerは
for job := range jobsでジョブを受け取り続け、jobsがcloseされて空になったらループを抜けてwg.Done() wg.Wait()を待つ専用のgoroutineが全worker完了を検知したらresultsをclose- メインgoroutineは
resultsをfor rangeで読み切る(③のcloseによってループが自然に終了する)
このうち1つでも順序を間違えると、デッドロック・パニック・結果の欠落のいずれかが起こります。
📝 問題
以下は「複数の注文の消費税込み金額をWorker Poolで並行計算する」ことを意図したコードです。一見動きそうに見えますが、実行すると必ずfatal error: all goroutines are asleep - deadlock!で異常終了します。
package main
import (
"fmt"
"sync"
"time"
)
type Order struct {
ID int
AmountYen int
}
type Result struct {
OrderID int
TaxedYen int
}
func worker(id int, jobs <-chan Order, results chan<- Result, wg *sync.WaitGroup) {
for job := range jobs {
time.Sleep(10 * time.Millisecond) // 税計算などの重い処理を模擬
results <- Result{OrderID: job.ID, TaxedYen: int(float64(job.AmountYen) * 1.1)}
}
wg.Done()
close(results)
}
func main() {
orders := []Order{
{ID: 1, AmountYen: 1000},
{ID: 2, AmountYen: 2000},
{ID: 3, AmountYen: 3000},
{ID: 4, AmountYen: 4000},
{ID: 5, AmountYen: 5000},
}
jobs := make(chan Order, len(orders))
results := make(chan Result)
var wg sync.WaitGroup
const numWorkers = 3
for i := 0; i < numWorkers; i++ {
wg.Add(1)
go worker(i, jobs, results, &wg)
}
for _, o := range orders {
jobs <- o
}
wg.Wait()
for r := range results {
fmt.Printf("order=%d taxed=%d\n", r.OrderID, r.TaxedYen)
}
}
課題
- このコードには独立した欠陥が3つ含まれています。それぞれを指摘し、「なぜその欠陥が問題を引き起こすのか」を実行時に何が起きるかのレベルで説明してください(単に「ここがおかしい」だけでなく、goroutineが何を待って止まっているかまで言葉にすること)。
- 3つの欠陥をすべて修正した完全なコードを書いてください。
runWorkerPool(orders []Order, numWorkers int) []Resultという関数に処理をまとめ、mainから呼び出す形にリファクタリングして構いません。 - 修正版に対して、以下を検証するテーブル駆動テストを書いてください。
- 5件の注文を渡すと、5件すべての結果が欠落なく返る(受け取った件数が注文件数と一致する。Worker Poolは並行実行のため到着順は保証されないので、
OrderIDでソートしてから期待値と比較すること) - 注文が0件のときは空の結果が返る(パニックしない)
numWorkersが注文件数より多い場合でも正しく全件処理される
- 5件の注文を渡すと、5件すべての結果が欠落なく返る(受け取った件数が注文件数と一致する。Worker Poolは並行実行のため到着順は保証されないので、
go test -raceで検証し、データ競合が検出されないことを確認してください。
go run . / go test -race -v ./...でそのまま動作する完全なコードとして提示すること。
🔍 ヒント(段階的開示)
ヒント1 — 方向性
3つの欠陥は、それぞれ「①いつchannelをcloseするか」「②誰がcloseするか」「③いつwg.Wait()するか」という異なる問いに対する誤りです。1つを直しても残り2つが残っていれば、症状(デッドロック/パニック/結果欠落)が変わるだけで壊れたままです。実際にgo run .で毎回同じfatal error: all goroutines are asleep - deadlock!が出ることを確認し、スタックトレースに現れるgoroutineが「どの行で・何を待って」止まっているかを1つずつ読んでみてください。
ヒント2 — アプローチ
- 欠陥③(最も直接的にデッドロックを引き起こしている箇所):
wg.Wait()をメインgoroutineの中で、resultsを読み始める前に呼んでいます。resultsはバッファなしchannelなので、workerは誰かが受け取ってくれるまで送信をブロックされます。しかしメインgoroutineは受け取りを始める前にwg.Wait()でブロックしており、workerの完了(wg.Done())はその送信が成功した後にしか来ません。→ お互いが相手を待ち続ける典型的なデッドロックです。wg.Wait()は別goroutineの中で行い、メインgoroutineは即座にresultsの受信を始める必要があります。 - 欠陥①:
jobsに全件送信したあと、close(jobs)を一度も呼んでいません。for job := range jobsは、channelがcloseされて中身が空になるまで終了しません。closeを忘れると、全ジョブを処理し終えたworkerは「まだ次のジョブが来るかもしれない」と永遠に待ち続けます(goroutineリーク)。 - 欠陥②:
close(results)を各workerの中で呼んでいます。3つのworkerがそれぞれ自分のforループを終えた瞬間にcloseしようとするため、2つ目以降のworkerは「すでに閉じたchannelを閉じようとする」ことになりpanic: close of closed channelを起こします(タイミングによっては、他のworkerがまだ結果を送ろうとしている最中にcloseされ、panic: send on closed channelになったり、メインの受信ループがその時点で終了してしまい残りの結果が静かに失われることもあります)。closeしてよいのは「全workerが終わったことを知っている」唯一の場所、つまりwg.Wait()の後だけです。
ヒント3 — コード骨格
func runWorkerPool(orders []Order, numWorkers int) []Result {
jobs := make(chan Order, len(orders))
results := make(chan Result, len(orders))
var wg sync.WaitGroup
for i := 0; i < numWorkers; i++ {
wg.Add(1)
go worker(i, jobs, results, &wg) // worker側は defer wg.Done() のみ。close(results)はしない
}
for _, o := range orders {
jobs <- o
}
close(jobs) // 欠陥①の修正: 送信側が送り終えたらjobsをclose
go func() {
wg.Wait()
close(results) // 欠陥②の修正: closeするのはここ1箇所だけ
}()
var out []Result
for r := range results { // 欠陥③の修正: wg.Wait()を待たずに即座に受信開始
out = append(out, r)
}
return out
}
✅ 模範解答
ファイル: main.go
package main
import (
"fmt"
"sync"
"time"
)
// Order は消費税計算対象の注文。
type Order struct {
ID int
AmountYen int
}
// Result はWorker Poolによる計算結果。
type Result struct {
OrderID int
TaxedYen int
}
// worker はjobsを受け取り続け、closeされて空になったらdefer wg.Done()で完了を通知する。
// results のcloseはここでは行わない(closeの責務は呼び出し元の1箇所に集約する)。
func worker(id int, jobs <-chan Order, results chan<- Result, wg *sync.WaitGroup) {
defer wg.Done()
for job := range jobs {
time.Sleep(10 * time.Millisecond) // 税計算などの重い処理を模擬
results <- Result{OrderID: job.ID, TaxedYen: int(float64(job.AmountYen) * 1.1)}
}
}
// runWorkerPool はnumWorkers個のworkerでordersを並行処理し、全結果を返す。
// 結果の順序は並行実行のため保証されない(呼び出し側がソート等で必要に応じて整える)。
func runWorkerPool(orders []Order, numWorkers int) []Result {
jobs := make(chan Order, len(orders))
results := make(chan Result, len(orders))
var wg sync.WaitGroup
for i := 0; i < numWorkers; i++ {
wg.Add(1)
go worker(i, jobs, results, &wg)
}
for _, o := range orders {
jobs <- o
}
close(jobs) // 送信側(唯一の送り手であるmain)がjobsを閉じる
// wg.Wait()は専用goroutineで行う。メインgoroutineをここでブロックすると、
// resultsの受信が始まらずworkerが送信できずデッドロックする。
go func() {
wg.Wait()
close(results) // 全worker完了を検知できる唯一の場所でresultsを閉じる
}()
var out []Result
for r := range results {
out = append(out, r)
}
return out
}
func main() {
orders := []Order{
{ID: 1, AmountYen: 1000},
{ID: 2, AmountYen: 2000},
{ID: 3, AmountYen: 3000},
{ID: 4, AmountYen: 4000},
{ID: 5, AmountYen: 5000},
}
results := runWorkerPool(orders, 3)
for _, r := range results {
fmt.Printf("order=%d taxed=%d\n", r.OrderID, r.TaxedYen)
}
}
ファイル: main_test.go
package main
import (
"sort"
"testing"
)
func TestRunWorkerPool(t *testing.T) {
orders := []Order{
{ID: 1, AmountYen: 1000},
{ID: 2, AmountYen: 2000},
{ID: 3, AmountYen: 3000},
{ID: 4, AmountYen: 4000},
{ID: 5, AmountYen: 5000},
}
results := runWorkerPool(orders, 3)
if len(results) != len(orders) {
t.Fatalf("len(results) = %d, want %d", len(results), len(orders))
}
// Worker Poolは並行実行のため到着順は保証されない。OrderIDでソートしてから比較する。
sort.Slice(results, func(i, j int) bool { return results[i].OrderID < results[j].OrderID })
for i, r := range results {
want := int(float64(orders[i].AmountYen) * 1.1)
if r.OrderID != orders[i].ID {
t.Fatalf("results[%d].OrderID = %d, want %d", i, r.OrderID, orders[i].ID)
}
if r.TaxedYen != want {
t.Fatalf("results[%d].TaxedYen = %d, want %d", i, r.TaxedYen, want)
}
}
}
func TestRunWorkerPool_EmptyOrders(t *testing.T) {
results := runWorkerPool(nil, 3)
if len(results) != 0 {
t.Fatalf("len(results) = %d, want 0", len(results))
}
}
func TestRunWorkerPool_MoreWorkersThanJobs(t *testing.T) {
orders := []Order{{ID: 1, AmountYen: 100}}
results := runWorkerPool(orders, 10)
if len(results) != 1 {
t.Fatalf("len(results) = %d, want 1", len(results))
}
}
▶ 実行結果を見る(go build / go vet / go test -race -v / go run で検証済み)
$ go vet ./... && go build ./...
(エラーなし)
$ go test -race -v ./...
=== RUN TestRunWorkerPool
--- PASS: TestRunWorkerPool (0.02s)
=== RUN TestRunWorkerPool_EmptyOrders
--- PASS: TestRunWorkerPool_EmptyOrders (0.00s)
=== RUN TestRunWorkerPool_MoreWorkersThanJobs
--- PASS: TestRunWorkerPool_MoreWorkersThanJobs (0.01s)
PASS
ok example.com/day029 2.024s
$ go run .
order=2 taxed=2200
order=1 taxed=1100
order=3 taxed=3300
order=5 taxed=5500
order=4 taxed=4400
(参考: 修正前のバグ入りコードを実際にgo run .すると、毎回下記のように再現性のあるデッドロックで落ちることを確認済み)
$ go run . # バグ入りコード(欠陥③が残った状態)
fatal error: all goroutines are asleep - deadlock!
goroutine 1 [sync.WaitGroup.Wait]:
sync.(*WaitGroup).Wait(...)
main.main()
.../main.go:51 +0x1dd
goroutine 18 [chan send]:
main.worker(...)
.../main.go:22 +0x85
(goroutine 19, 20も同様に chan send で停止)
exit status 2
さらに、欠陥③だけを直して欠陥①②を残したまま実行すると、close(results)を複数workerが試みることで結果が5件中3件しか出力されずに正常終了してしまう(パニックすら起きない)ケースも実機で確認しています。これはタイミング次第でpanic: send on closed channelにもなり得る、再現性の低い危険なバグです。
go.mod・main.go・main_test.goを1モジュール(module example.com/day029)として配置し、go vet / go build / go test -race -v ./... / go run .で実際に検証済みです。
🪜 Step-by-Step 解説
goroutine 1 [sync.WaitGroup.Wait]: // メインgoroutineがwg.Wait()で停止
goroutine 18 [chan send]: // workerがresults<-で送信をブロックされている
Goランタイムは全goroutineが眠り込んで誰も進めなくなるとfatal error: all goroutines are asleep - deadlock!を出して即座にプロセスを終了させます。このメッセージと各goroutineの停止位置([sync.WaitGroup.Wait]や[chan send])を読むだけで、「メインはwg.Wait()待ち」「workerは送信待ち」という循環待ちの構造がそのまま見えます。
go func() {
wg.Wait()
close(results)
}()
for r := range results { // メインは即座にここへ来る
メインgoroutineの役目を「resultsを読み切ること」に一本化し、「全worker完了の検知」は別goroutineに任せます。これにより、メインが受信を始めるのを妨げるものが何もなくなり、循環待ちが解消されます。
close(jobs) // mainが送信し終えた直後(送信側の責務)
close(results) // wg.Wait()の直後(全workerの完了を知っている唯一の場所の責務)
jobsの送り手はmainだけなので、mainがcloseするのは自然です。一方resultsには3つのworkerが書き込んでおり、「誰か1人が終わったらclose」では他の2人がまだ書き込み中かもしれません。「全員が終わったことを知っている場所」はwg.Wait()が返ってきた地点しかなく、そこにcloseの責務を1本化する必要があります。
func worker(id int, jobs <-chan Order, results chan<- Result, wg *sync.WaitGroup) {
defer wg.Done()
for job := range jobs { ... }
}
wg.Done()をループの後に素朴に書くと、ループ内で予期しないpanicが起きた場合にwg.Done()が呼ばれず、wg.Wait()が永久に返らなくなります。deferにしておけば、正常終了・panic・どちらの経路でも確実に呼ばれます。
💡 設計思想・なぜこう書くのか
wg.Wait()直後の専用goroutine)で作る必要があります。これはGoの「明示性を好む」思想の延長で、コンパイラが守ってくれない規律をコードの形で可視化しているとも言えます。fatal errorで落としてくれます。「テストがたまたま通った」で済ませず、go runを複数回試して再現性を確認する癖が、並行処理のデバッグでは特に重要です。🌐 他言語との比較
| 観点 | Go(channel + WaitGroup) | Java(ExecutorService) | Python(concurrent.futures.ThreadPoolExecutor) | Node.js(worker_threads / piscina) |
|---|---|---|---|---|
| Worker数の制御 | numWorkers個のgoroutineを自前でforループ起動 | Executors.newFixedThreadPool(n) | ThreadPoolExecutor(max_workers=n) | piscina等のライブラリでmaxThreads指定 |
| ジョブの投入 | jobschannelへの送信 | executor.submit(task) | executor.submit(fn, arg) | pool.run(data) |
| 完了待ち・結果集約 | sync.WaitGroup + 専用channel | Future.get() / invokeAll() | as_completed() / Future.result() | Promise.all() |
| 「閉じ忘れ」系バグの起きやすさ | channelのclose責務を自分で設計する必要があり、今回のような欠陥が起きやすい | プールのshutdown()忘れはあるが、closeの所有権問題は言語機構としては存在しない | 同上(with文でコンテキスト管理すれば自動close) | Promiseベースのため、close忘れよりは未処理rejectionの方が典型的な事故 |
Goが「自分でchannelのcloseを設計させる」のは不便に見えますが、その分「誰が・いつ・何をcloseするか」を完全にコードで説明できる透明性があります。Java/Pythonのスレッドプールはこの部分をランタイム側が隠蔽してくれる代わりに、内部で何が起きているかをGoほど細かくは制御できません。
🏆 実務での使いどころ
- バッチ処理・大量データの並行変換: CSVの数万行を並行してパース・変換する処理はWorker Poolの典型的な使いどころです
- 外部API呼び出しの並行化: Day 028で学んだ
HTTPStockCheckerのようなHTTP呼び出しを、複数のSKUに対して同時に(かつnumWorkersで上限を切って)叩く場合にそのまま応用できます - 画像・動画のリサイズ/エンコードパイプライン: CPU負荷の高い処理を「CPUコア数程度」のworkerに制限しながら並行実行する場面で使われます
- Fan-in/Fan-out・Pipelineパターンへの発展: Day 030以降で扱う予定のFan-in/Fan-outやPipelineパターンは、今日のWorker Poolを複数段つなげたものとして理解できます
⚠️ よくある誤解・ミス
| 誤解・ミス | なぜ起こるか | 正しい理解 |
|---|---|---|
wg.Wait()をメインgoroutineで、結果の受信より先に呼んでしまう | 「全部終わってから結果を集めればいい」という直感的な順序で書いてしまう | resultsがバッファなし(または小さいバッファ)の場合、workerは受信されるまで送信をブロックされる。受信を始める前に完了を待つと循環待ちになる |
| 複数のgoroutineが同じchannelをそれぞれcloseしようとする | 「自分の仕事が終わったから閉じよう」と各goroutineが独立に判断してしまう | closeは「全員の完了を知っている唯一の場所」だけの責務。2回目のcloseはpanic: close of closed channelになる |
送信側のchannel(jobs)をcloseし忘れる | 受信側のrangeが自動で終わると思い込んでしまう | for range channelはcloseされるまで終了しない。全送信後に明示的にclose()しないとworkerがリークする |
wg.Done()をループの外に素朴に書き、panic時に呼ばれないことに気づかない | 正常系だけを想定してコードを書いてしまう | defer wg.Done()にしておけば、途中でpanicしてもgoroutineの完了はWaitGroupに正しく伝わる |
| バグが「たまに」しか再現しないと「まあ動いている」と判断してしまう | 数回go runして問題が出なかったので安心してしまう | 今回の欠陥②のように、タイミング次第でパニックしたり結果が静かに欠落したりするバグは、複数回実行して再現性を確認しないと見逃す。go test -raceを習慣的に回すことが実務での最低限の防御線になる |
🚀 次のステップ
- 発展:
resultsチャネルのバッファサイズを0(今回のバグ版と同じ)に戻した状態で修正版のロジック(wg.Wait()を専用goroutineに退避する形)を使うとどうなるか、実際に試してみましょう。バッファの有無が処理の速度やブロックの仕方にどう影響するかが体感できます - 次回予告: Day 030 — 並行処理パターン②(Fan-in/Fan-out)(実装)。今日のWorker Poolを土台に、複数のソースからの結果を1つのchannelに集約する「Fan-in」と、1つの入力を複数の処理系統に分配する「Fan-out」を実装します