Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Performance] Probème N+1 sur la page etablissements #50

Closed
bdahon opened this issue Mar 30, 2020 · 4 comments
Closed

[Performance] Probème N+1 sur la page etablissements #50

bdahon opened this issue Mar 30, 2020 · 4 comments
Labels
scaling A considérer si national

Comments

@bdahon
Copy link
Collaborator

bdahon commented Mar 30, 2020

La page qui liste les établissement présente un gros problème de requête n+1. Pour chaque établissement dans la page, plusieurs requêtes sont exécutées depuis le render de la page

Voici ce que l'on a quand on loggue toutes les requêtes sur cette page.
image

On voit donc qu'on peut dépasser les 20 requêtes juste pour afficher 10 éléments.
Il faut retravailler le controller pour qu'il crée un value object plutôt que de renvoyer le modèle directement.

@bdahon bdahon added the scaling A considérer si national label Mar 30, 2020
@bdahon bdahon added this to the Mise en production milestone Mar 30, 2020
@rleger
Copy link
Owner

rleger commented Mar 31, 2020

Tout à fait, j'ai voulu m'y attaquer il y a longtemps.. mais manque de temps.
il compile pas mal de trucs mais il faut l'optimiser c'est sûr.. pour garder les fonctions de pagination c'est un peu de boulot non ?

@bdahon
Copy link
Collaborator Author

bdahon commented Mar 31, 2020

Est-ce que l'on est vraiment obligé de garder la pagination ? On va avoir au niveau national une centaine d'établissements et la carte récupère toutes les infos quoiqu'il arrive.

@rleger
Copy link
Owner

rleger commented Mar 31, 2020

Oui bonne remarque c'est vrai..

@bdahon
Copy link
Collaborator Author

bdahon commented Mar 31, 2020

Si j'ai un peu de temps ce matin, je regarde.

rleger added a commit that referenced this issue Apr 3, 2020
Fix issue #50: [Performance] Problème N+1 sur la page etablissements
@rleger rleger closed this as completed Apr 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scaling A considérer si national
Projects
None yet
Development

No branches or pull requests

2 participants