Remove double buffer in http.IncomingMessage

Aperta
#148 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
25/100
Tipo di issue
Refactoring
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
node.js
Ambito
backend, performance

Direzione di ricerca

Inizia esaminando l’issue #147 e l’implementazione di http.IncomingMessage, Readable e del buffering del socket coinvolto. Determina se sia possibile rimuovere il buffering di Readable senza introdurre una breaking change inaccettabile; il lavoro è completato quando l’overhead è ridotto e l’impatto sulla compatibilità è stato risolto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

http.IncomingMessage is an instance of Readable, which does buffering. However the data is coming from a socket, which also does buffering. If we could remove the Readable buffering, we could remove quite a bit of overhead.

This might be too breaking.

#147

Lingua principale
Shell
Stelle
399
Fork
11
Merge medio
29m
PR unite (30g)
1

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di nodejs/performance

Tutte le issue di nodejs/performance

Issue simili

Altre issue su Shell/Bash

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.