dotnet/runtime

TakeLast Performance Improvement suggestion

Open

#118.312 aperta il 3 ago 2025

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-System.Linqhelp wantedtenet-performance

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

TakeRangeFromEndIterator uses a regular Queue <T>when IsStartIndexFromEnd, but this process can be done without intervening yields, allowing optimization using ArrayPool and Span.

https://github.com/dotnet/runtime/blob/3f25a276772b8d0e54452321e7525b7a176aa4a7/src/libraries/System.Linq/src/System/Linq/Take.cs#L111C13-L154C18

Below is the PR I did on ZLinq. https://github.com/Cysharp/ZLinq/pull/207/files Unlike ZLinq, a ToArray is required at the end of the queueing for Linq, but it will be similar.

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente di TakeRangeFromEndIterator in Take.cs (riga 111). Rivedi la PR collegata in ZLinq (https://github.com/Cysharp/ZLinq/pull/207) per i pattern di ottimizzazione che utilizzano ArrayPool e Span. Indaga se ToArray è necessario dopo l'accodamento e se lo stesso approccio può essere applicato senza introdurre modifiche di rottura. Considera i benchmark delle prestazioni e i compromessi nell'allocazione della memoria.
Tech stack
csharp
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C#LINQArrayPoolSpan
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30