pingcap/tidb

Eliminate unnecessary `PessimisticRollback` when write conflict occurs

Open

#17.597 geöffnet am 2. Juni 2020

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Go (6.186 Forks)batch import
challenge-programhelp wantedsig/transactiontype/enhancement

Repository-Metriken

Stars
 (40.090 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 14T 4h) (346 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Verstehe den pessimistischen Sperrerwerbsablauf in TiDB und TiKV, identifiziere, wann PessimisticRollback unnötig gesendet wird, und ändere den Code, um das Senden des Rollbacks zu überspringen, wenn ein Schreibkonflikt auftritt und Sperren für einen erneuten Versuch aufbewahrt werden können.
Tech Stack
go
Domain
backenddatabase
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Über 1 Woche
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30