dotnet/runtime

ServiceController.WaitForStatus could use NotifyServiceStatusChange WinAPI to avoid polling

Open

#45 049 ouverte le 21 nov. 2020

Voir sur GitHub
 (0 commentaires) (2 réactions) (0 assignés)C# (5 445 forks)batch import
area-System.ServiceProcesshelp wanted

Métriques du dépôt

Stars
 (17 886 stars)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez la fonction WinAPI NotifyServiceStatusChange et comprenez comment l'utiliser pour remplacer la boucle de polling dans ServiceController.WaitForStatus. Consultez la PR liée et les commentaires pour le contexte, puis implémentez la modification avec une gestion appropriée des erreurs et des considérations multiplateformes (le cas échéant).
Stack technique
csharp
Domaine
backendapiperformance
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C#Windows API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35