Refactor `listTemplates` and `listIsos` workflows
- Vorherrschende Sprache
- Java
- Sterne
- 3.1k
- Forks
- 1.4k
- Ø Merge
- 6 T. 19 Std.
- Gemergte PRs (30 T.)
- 32
Beschreibung
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
Beitragsleitfaden
Rechercherichtung
Beginne in server/src/main/java/com/cloud/api/query/QueryManagerImpl.java bei searchForTemplatesInternal und verfolge anschließend die Workflows listIsos und listTemplates sowie deren direkte DAO-Operationen. Das Refactoring ist abgeschlossen, wenn die Workflows modularisiert, die Verantwortlichkeiten der Service-Schicht verbessert, Logging hinzugefügt und Unit-Tests erstellt wurden, die das geänderte Verhalten abdecken.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- backend-api-design, cloud
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100