sindresorhus/del

Add default concurrency limit

Open

#60 geöffnet am 9. Juni 2017

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.305 Stars) (71 Forks)batch import
enhancementhelp wanted

Beschreibung

Currently the concurrency limit is Infinity. I have a gut feeling it would generally be faster with some kind of concurrency limit. Probably a number between 4 and 10. This will need some benchmarking. Help wanted.

Context: https://github.com/sindresorhus/del/commit/66f069f1dedb7d077bc35b9d6c50f278418ff34c#commitcomment-21722984

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backendcli
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Node.js asyncbenchmarkingconcurrency concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45
Research-Richtung
Investigate current concurrency implementation in the del repo (likely in the main module). Run benchmarks with different concurrency limits (4-10) to compare performance against Infinity. Decide on a default value based on results, then implement the change and ensure existing tests pass. Consider adding a configurable option as well.