99designs / 99designs/cmdstalk

Performance: The run loop calls stats-job multiple times

Aberta
#16 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Go
Estrelas
76
Forks
14
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

For every loop of processing a job, there are multiple consecutive implicit calls to https://github.com/99designs/cmdstalk/blob/fbd704da919d283643e673bd02efcf0221224b92/bs/job.go#L93. This requires a full , unnecessary, roundtrip on the network.

To fix this, [the Run loop](https://github.com/99designs/cmdstalk/blob/fbd704da919d283643e673bd02efcf0221224b92/broker/broker.go#L97) could extract the stats for a job once, and parse the necessary stats.

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

A issue aponta para broker/broker.go, linha 97, referente ao loop Run, e para bs/job.go, linha 93, referente à chamada stats-job. Comece lendo o método Run do broker para entender a estrutura do loop e, em seguida, examine como as estatísticas do job são obtidas e usadas. A correção envolve armazenar em cache as estatísticas de uma única chamada e fazer o parsing delas dentro do loop. Verifique os testes existentes no pacote broker para garantir que as alterações não quebrem a funcionalidade existente.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Domínio
backend, performance
Tipo de issue
Refatoração
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
50/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.