search

MTES-MCT / dialog

Intégration de la réglementation de circulation dans les solutions numériques
https://dialog.beta.gouv.fr
GNU Affero General Public License v3.0
8 stars 0 forks source link

Optimise les requêtes de la page de détail #852

Closed florimondmanca closed 4 days ago

florimondmanca commented 5 days ago

On passe de 5 + 2*N + P requêtes (où N = le nb de localisations et P = le nb de périodes) à un nb fixe de 5 requêtes :+1:

Il manquait la présence de dailyRange, namedStreet et numberedRoad dans le addSelect() de la requête qui récupère les mesures. Donc l'ORM faisait les requêtes à la volée.

J'ai aussi remplacé la requête qui récupérait l'arrêté entier (alors qu'on récupère les mesures en parallèle) par une requête qui récupère seulement ce qu'il nous fallait : l'ID de son organisation

Confirmation : en intégrant cette PR ainsi que l'ajout d'une jointure sur l.rawGeoJSON à #847, j'y ai bien aussi un nb fixe de 5 requêtes (au lieu de 1297 en l'état !)

codecov-commenter commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.11%. Comparing base (9840c8f) to head (a7d4673). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #852 +/- ## ========================================= Coverage 99.10% 99.11% - Complexity 1170 1179 +9 ========================================= Files 218 221 +3 Lines 4939 4969 +30 ========================================= + Hits 4895 4925 +30 Misses 44 44 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

florimondmanca commented 4 days ago

@mmarchois Tests ajoutés, query renommée