pingcap/tidb

Move the check for `_tidb_rowid` from executor to planner

Open

#14,221 opened on 2019年12月25日

GitHub で見る
 (0 comments) (0 reactions) (1 assignee)Go (6,186 forks)batch import
help wantedtype/enhancement

Repository metrics

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

説明

Feature Request

Describe the feature you'd like:

Currently we check the _tidb_rowid in executor: https://github.com/pingcap/tidb/blob/57ecfb9b466392853200e9ffebcdd6ebf631ab91/executor/update.go#L284-L293 https://github.com/pingcap/tidb/blob/57ecfb9b466392853200e9ffebcdd6ebf631ab91/executor/insert_common.go#L135-L142

It's better to check this in planer.

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

調査方針
executor/update.goとexecutor/insert common.goの2箇所で tidb rowidがチェックされている場所を調べてください。プランナが現在この制約をどのように処理しているかを理解し、チェックをプランナの論理最適化フェーズに移動してください。
技術スタック
go
領域
backend
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GoTiDB execution and planning concepts
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70