pingcap/tidb

use hack.Slice in ColumnInfo:Dump to avoid unnecessary `string to slice` conversation

Chiusa

#32.073 aperta il 30 gen 2022

Vedi su GitHubGuida contributor
 (3 commenti) (1 reazione) (1 assegnatario)Go (6186 fork)batch import
help wantedtype/enhancementtype/performance

Metriche repository

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

Descrizione

Enhancement

In some read-heavy workloads, we found that Column Dump causes some CPUs

image

Through the flamegraph, we can see that nearly 1% of CPU is caused by string to slice conversation. We can ensure that we don't change the slice value in the Dump function, https://github.com/pingcap/tidb/blob/master/server/column.go#L41, so we can use hack.Slice instead.

Guida contributor

Direzione di ricerca
Sostituisci la conversione da stringa a []byte con hack.Slice nella funzione Dump di ColumnInfo in server/column.go per migliorare le prestazioni.
Tech stack
go
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80