tikv/tikv

Add Box<> to some types to eliminate move cost

Aperta

#4042 aperta il 9 gen 2019

Vedi su GitHubGuida contributor
 (4 commenti) (1 reazione) (0 assegnatari)Rust (2189 fork)batch import
component/performancehelp wantedstatus/mentor

Metriche repository

Star
 (15.922 stelle)
Metriche merge PR
 (Merge medio 28g 9h) (68 PR mergiate in 30 g)

Descrizione

Feature Request

Some types' size is large, e.g. mysql::Time has a size of about 50 bytes. When variables in these types are moved, it costs a lot. Additionally, Datum is an enum including a mysql::Time so that Datum becomes this large as well!

By wrapping such type in a Box we may eliminate some moving cost.

Notice that we should only do this in performance critical path.

Guida contributor

Direzione di ricerca
Identifica i tipi grandi nei percorsi critici per le prestazioni e avvolgili in Box<...> per ridurre i costi di spostamento. Usa profiling o revisione del codice per trovare candidati.
Tech stack
rust
Dominio
backenddatabase
Tipo issue
Performance
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.
Abbastanza chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70