apache / apache/cloudstack

Refactor `listTemplates` and `listIsos` workflows

Aberta
#11,209 1 comentário 1 reação 0 responsáveis Ver no GitHub
type:cleanup type:enhancement
Linguagem predominante
Java
Estrelas
3.1k
Forks
1.4k
Merge médio
6d 19h
PRs com merge (30d)
32

Descrição

The workflows of the `listIsos` and `listTemplates` APIs, particularly the `com.cloud.api.query.QueryManagerImpl#searchForTemplatesInternal` method, could greatly benefit from refactoring.

This method accepts over 20 parameters, spans approximately 200 lines, performs DAO operations directly from the business/service layer, and lacks modularization, logging, and unit tests. These issues make troubleshooting and maintenance challenging.

https://github.com/apache/cloudstack/blob/f52e05863e608f53f9e97e93fe47577b8dfe189b/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java#L4802-L5007

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece em server/src/main/java/com/cloud/api/query/QueryManagerImpl.java, em searchForTemplatesInternal, e depois acompanhe os workflows listIsos e listTemplates e suas operações DAO diretas. A refatoração estará concluída quando os workflows forem modularizados, as responsabilidades da camada de serviço forem aprimoradas, o logging for adicionado e os testes unitários cobrirem o comportamento alterado.

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

Avaliação

Stack de tecnologia
java
Domínio
backend-api-design, cloud
Tipo de issue
Refatoração
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

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