pingcap/tidb

Proposal to use atomic.Pointer[T] instead of atomic.Value

Open

#44.736 aperta il 16 giu 2023

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)Go (6186 fork)batch import
good first issuetype/enhancement

Metriche repository

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

Descrizione

Enhancement

Currently, the usage of atomic.Value is prevalent in our codebase. However, I would like to propose using atomic.Pointer[T] instead. The reason being that atomic.Pointer[T] provides a clearer expression of the type, which can help in reducing type conversions.

Therefore, I suggest that we consider using atomic.Pointer[T] in our codebase wherever possible. This change will not only result in better code clarity but also improve the performance of our code.

Guida contributor

Direzione di ricerca
Identifica tutti gli usi di atomic.Value nel codebase, determina il tipo concreto T per ciascuno e sostituisci con atomic.Pointer[T] assicurando nessun cambiamento funzionale.
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.
Mezza giornata
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.
50