pingcap/tidb

executor.Next() should not call chid.Open() or child.Close()

Open

#4,983 opened on 2017年11月2日

GitHub で見る
 (1 comment) (0 reactions) (1 assignee)Go (6,186 forks)batch import
help wanted

Repository metrics

Stars
 (40,090 stars)
PR merge metrics
 (平均マージ 14d 4h) (30d で 346 merged PRs)

説明

calling Open() or Close() during Next() makes our executors un-maintainable, for example, if we comment out the following lines then make test, our ut failes ... https://github.com/pingcap/tidb/blob/0616a2fd1511e2206086c212261d93b1106605c7/executor/executor.go#L345

or https://github.com/pingcap/tidb/blob/0616a2fd1511e2206086c212261d93b1106605c7/executor/join.go#L88

Open is used to initialize executors and allocate resources for every executor Close is used to deallocate the resource adopted by every executor, avoid memory leak

コントリビューターガイド

調査方針
executor/executor.go および executor/join.go 内の参照を調べて、現在のパターンを理解します。Next 契約に違反せずに Open および Close を呼び出す適切な場所を特定します。問題で指摘されているように、現在のコードではテストが失敗するため、リファクタリング後も既存のテストが合格することを確認します。一貫性を保つために、より広範なエグゼキュータ階層を考慮してください。
技術スタック
go
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
Go
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60