graphql/dataloader

Ever considered the use of `queueMicrotask`?

Open

#348 geöffnet am 17. Aug. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (12.520 Stars) (556 Forks)batch import
help wanted

Beschreibung

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.

Contributor Guide

Tech Stack
javascriptnodejs
Domain
performance
Issue Type
research
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
needs maintainer response
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.jsJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.