golang/go

internal/singleflight: delete, use golang.org/x/sync/singleflight

Aperta

#31.697 aperta il 26 apr 2019

Vedi su GitHubGuida contributor
 (13 commenti) (4 reazioni) (0 assegnatari)Go (19.008 fork)batch import
NeedsFixhelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The singleflight package started life in internal/singleflight and then moved to golang.org/x/sync/singleflight, and the two have had slightly divergent development histories since. It's getting kinda messy. Their APIs have even somewhat diverged.

I just tried to re-unify them and delete internal/singleflight from std, but we can't use mod/vendor packages during bootstrap.

/cc @ianlancetaylor

Guida contributor

Direzione di ricerca
Esamina le differenze tra le API e le implementazioni di internal/singleflight e golang.org/x/sync/singleflight. Studia come il processo di bootstrap di Go impedisce l'uso di moduli esterni durante il bootstrap. Proponi un piano per consolidare i due pacchetti senza rompere il bootstrap, possibilmente copiando l'implementazione esterna in internal o usando tag di build.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Abbastanza chiara
Prerequisiti
Gounderstanding of Go modulesGo bootstrap process
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40