Perf - Add buffer pooling where relevant

Aperta

#832 aperta il 25/4/2023 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 5/5
Tempo stimato: Più di una settimana
Idoneità per principianti: 35/100

Tipo di issue: Funzionalità
Chiarezza: Da chiarire
Stato di attività: Ferma
Stack tecnologico: csharp
Ambito: performance

Direzione di ricerca

Esamina InflaterInputStream e le classi di compressione correlate che allocano buffer nei loro costruttori; confronta le alternative proposte caller-supplied-buffer e ArrayPool. Definisci l’ambito, la proprietà dei buffer e il comportamento di rilascio prima dell’implementazione, quindi verifica che una compressione intensiva riduca la pressione sulle allocazioni senza modificare il comportamento dello stream.

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

Descrizione

enhancement performance

Current situation

Some pieces of code such as InflaterInputStream, through InflaterInputStream, allocate buffers upfront in their constructors, with no way to control this behavior (e.g : passing the buffer to use).
In code making intensive use of such classes (e.g : app sending huge amounts of compressed data in my case), this can result in this being unsustainable in term of resulting GC load.

Describe the solution you'd like

I would like to suggest some alternatives :

add constructors overload admitting the buffer to use (so client can handle the reuse logic)
use ArrayPool all the time, this is choice taken by Microsoft in DeflateStream. While it allocates a new buffer on Net Fx (https://referencesource.microsoft.com/#System/sys/System/IO/compression/DeflateStream.cs,63), on Net Core it is always retrieved from the Shared pool and returned when the stream is disposed (https://source.dot.net/#System.IO.Compression/System/IO/Compression/DeflateZLib/DeflateStream.cs,109)
any mix of the two previous solutions.

Describe alternatives you've considered

Due to current design of most classes there is sadly no alternative as there is no control over the buffers allocations in constructors.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

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

Altre issue di icsharpcode/SharpZipLib

docs/specification/appnote.txt is non-distributable Aperta

*no response* bug

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100

icsharpcode/SharpZipLib#905 · 1 commento · Aperta il 14/4/2026
BZip2InputStream: multiple crashes from malformed input Aperta

bug bzip2

Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100

icsharpcode/SharpZipLib#904 · Aperta il 15/3/2026
SetLevel in ZipFile Aperta

enhancement zip

Difficoltà 2/5 1-2 giorni Idoneità per principianti 55/100

icsharpcode/SharpZipLib#903 · Aperta il 23/12/2025
.z file decompress throw exception, header byte 0x1f, 0x9d, 0x90,0x23, winrar and 7z tools open success. Aperta

*no response* bug

Difficoltà 4/5 3-5 giorni Idoneità per principianti 32/100

icsharpcode/SharpZipLib#901 · 1 commento · Aperta il 2/11/2025
Extraction of password-protected Zip files failed. Aperta

*no response* bug

Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100

icsharpcode/SharpZipLib#894 · 1 commento · Aperta il 14/5/2025