99designs / 99designs/cmdstalk
Performance: The run loop calls stats-job multiple times
- 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