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 (71 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (1.305 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

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

Research-Richtung
Untersuche die aktuelle Dateilöschschleife und füge einen Nebenläufigkeitsbegrenzer hinzu (z.B. mit p limit). Führe Benchmarks mit verschiedenen Grenzen (4-10) durch, um die optimale Leistung zu finden.
Tech Stack
javascriptnodejs
Domain
tooling
Issue Type
Funktion
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65