dotnet/runtime

ServiceController.WaitForStatus could use NotifyServiceStatusChange WinAPI to avoid polling

Open

#45.049 aperta il 21 nov 2020

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-System.ServiceProcesshelp wanted

Metriche repository

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

Descrizione

Currently, ServiceController WaitForStatus method polls every 250ms to check if the service has reached the desired status. This is not optimal and also not very performant, particularly if the service takes less than 250ms to reach that status. Apparently lost in the mists of time, there was a plan to use the WinAPI function NotifyServiceStatusChange to avoid polling and get instantly notified when the service reaches a specific status.

I'm opening this issue to prevent this potential improvement from getting lost again.

Props to @KalleOlaviNiemitalo for finding this.

Guida contributor

Direzione di ricerca
Studia la funzione WinAPI NotifyServiceStatusChange e capisci come usarla per sostituire il loop di polling in ServiceController.WaitForStatus. Guarda la PR collegata e i commenti per contesto, quindi implementa la modifica con una corretta gestione degli errori e considerazioni multipiattaforma (se presenti).
Tech stack
csharp
Dominio
backendapiperformance
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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#Windows API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35