pingcap/tidb

Eliminate unnecessary `PessimisticRollback` when write conflict occurs

Open

#17.597 aperta il 2 giu 2020

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)Go (6186 fork)batch import
challenge-programhelp wantedsig/transactiontype/enhancement

Metriche repository

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

Descrizione

Description

When TiDB fails to acquire pessimistic locks due to write conflict, it sends PessimisticRollback to release all locks the DML needs to lock because it's possible some keys needn't be locked after retry. However, the probability is small and PessimisticRollback brings overhead:

  • TiKV process more requests and write more locks.
  • A transaction releases all its locks and if it needs to lock them again, maybe some locks are held by other ones.

Score

  • 4000

SIG Slack Channel

You can join #sig-transaction on slack in your spare time to discuss and get help with mentors or others.

Mentor(s)

  • @youjiali1995

Contact the mentors: #tidb-challenge-program channel in TiDB Community Slack Workspace

Recommended Skills

  • Golang
  • TiDB
  • Pessimistic transaction

Learning Materials

Guida contributor

Direzione di ricerca
Comprendi il flusso di acquisizione del blocco pessimistico in TiDB e TiKV, identifica quando PessimisticRollback viene inviato inutilmente e modifica il codice per saltare l'invio del rollback quando si verifica un conflitto di scrittura e i blocchi possono essere mantenuti per il ritentativo.
Tech stack
go
Dominio
backenddatabase
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Oltre 1 settimana
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
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30