pingcap/tidb

Move the check for `_tidb_rowid` from executor to planner

Open

#14.221 aperta il 25 dic 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Go (6186 fork)batch import
help wantedtype/enhancement

Metriche repository

Star
 (40.090 star)
Metriche merge PR
 (Merge medio 14g 4h) (346 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Ispeziona le due posizioni in executor/update.go e executor/insert common.go in cui viene controllato tidb rowid. Comprendi come il planner gestisce attualmente questo vincolo e sposta il controllo nella fase di ottimizzazione logica del planner.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoTiDB execution and planning concepts
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70