dotnet/runtime

ServiceController.WaitForStatus could use NotifyServiceStatusChange WinAPI to avoid polling

Open

#45.049 aberto em 21 de nov. de 2020

Ver no GitHub
 (0 comments) (2 reactions) (0 assignees)C# (5.445 forks)batch import
area-System.ServiceProcesshelp wanted

Métricas do repositório

Stars
 (17.886 stars)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue a função WinAPI NotifyServiceStatusChange e entenda como usá la para substituir o loop de polling em ServiceController.WaitForStatus. Veja o PR vinculado e os comentários para contexto, depois implemente a alteração com tratamento adequado de erros e considerações de plataforma cruzada (se houver).
Pilha de tecnologia
csharp
Domain
backendapiperformance
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C#Windows API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35