dotnet/runtime

Add triggering test for GC.AddMemoryPressure/GC.RemoveMemoryPressure

Open

#36.796 geöffnet am 21. Mai 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-GC-coreclrhelp wantedincrease-code-coverage

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

The underlying GC.AddMemoryPressure/GC.RemoveMemoryPressure implementations were updated in https://github.com/dotnet/runtime/pull/36767. The existing testing was questionable already and does nothing for this other algorithm. Deferring adding equivalent testing for now since this code path is not new but has been around since .NET Framework.

https://github.com/dotnet/runtime/blob/4eaccd61b1a2a4b078140f8b36f2b45ab5980e8e/src/coreclr/tests/src/GC/API/GC/AddMemoryPressureTest.cs#L136-L159

/cc @VSadov

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandene Datei AddMemoryPressureTest.cs und die Änderungen aus PR #36767, um den neuen GC Algorithmus zu verstehen, und schreiben Sie dann einen Test, der das Verhalten von GC.AddMemoryPressure und GC.RemoveMemoryPressure unter verschiedenen Speicherdruckwerten überprüft.
Tech Stack
csharp
Domain
backend
Issue Type
Test
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
C#.NETGC concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65