graphql/dataloader

Ever considered the use of `queueMicrotask`?

Open

#348 aperta il 17 ago 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (556 fork)batch import
help wanted

Metriche repository

Star
 (12.520 star)
Metriche merge PR
 (Merge medio 5g 2h) (1 PR mergiata in 30 g)

Descrizione

It's available as of v11—isn't it better than process.nextTick? https://nodejs.org/api/globals.html#queuemicrotaskcallback

https://github.com/graphql/dataloader/blob/d336bd15282664e0be4b4a657cb796f09bafbc6b/src/index.js#L239-L255

Just a question...

Thanks, A.

Guida contributor

Direzione di ricerca
Confronta queueMicrotask e process.nextTick in Node.js e valuta se sia appropriato sostituire process.nextTick con queueMicrotask nello scheduler di DataLoader (src/index.js righe 239-255), considerando le differenze di temporizzazione, gestione degli errori e compatibilità.
Tech stack
javascriptnodejs
Dominio
backendapi
Tipo issue
Ricerca
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80