graphql/dataloader

Ever considered the use of `queueMicrotask`?

Open

#348 ouverte le 17 août 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (12 520 stars) (556 forks)batch import
help wanted

Description

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.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
performance
Type d'issue
research
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
needs maintainer response
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate the differences between queueMicrotask and process.nextTick in Node.js. Review the DataLoader source code at src/index.js lines 239-255 to understand the current usage of process.nextTick. Check the Node.js documentation for queueMicrotask and determine if it is a suitable replacement. Consider any compatibility issues with older Node.js versions. Look at the issue comments for any maintainer feedback. If deemed appropriate, prepare a pull request replacing process.nextTick with queueMicrotask.