apache / apache/cloudstack

Refactor `listTemplates` and `listIsos` workflows

Ouverte
#11,209 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub
type:cleanup type:enhancement
Langage dominant
Java
Étoiles
3.1k
Forks
1.4k
Merge moyen
6 j 19 h
PR mergées (30 j)
32

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez dans server/src/main/java/com/cloud/api/query/QueryManagerImpl.java, au niveau de searchForTemplatesInternal, puis suivez les workflows listIsos et listTemplates ainsi que leurs opérations DAO directes. Le refactoring est terminé lorsque les workflows sont modularisés, que les responsabilités de la couche service sont améliorées, que le logging est ajouté et que des tests unitaires couvrent le comportement modifié.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
backend-api-design, cloud
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.