Skip to content

#4765 [ActionPlan] fix: eliminate N+1 queries on kanban list#4766

Open
evarisk-kilyan wants to merge 1 commit into
developfrom
fix/4765-actionplan-list-perf
Open

#4765 [ActionPlan] fix: eliminate N+1 queries on kanban list#4766
evarisk-kilyan wants to merge 1 commit into
developfrom
fix/4765-actionplan-list-perf

Conversation

@evarisk-kilyan
Copy link
Copy Markdown
Contributor

Changements

  • Suppression du re-fetch redondant de SaturneTask dans la boucle (l'objet t etait deja charge)
  • Remplacement du User::fetch() par tache pour les photos : le champ photo est deja retourne par liste_contact()
  • Remplacement du Risk::fetch() x N par un seul Risk::fetchAll() pour tous les risques uniques
  • Remplacement du RiskAssessment::fetchAll() x N par un seul appel sur tous les risques
  • Remplacement du User::fetch() x N pour les evaluateurs par une seule requete SQL scopee aux IDs necessaires
  • Remplacement du COUNT SQL par tache pour les fichiers par une seule requete GROUP BY avant la boucle
  • Remplacement de Categorie::containing() x N par une seule requete IN avec JOIN
  • Scope de la requete projet_task_extrafields aux task IDs du projet courant

Impact performance

Avec 50 taches et 2 contacts par tache : ~350 requetes SQL -> ~10 requetes fixes independantes du nombre de taches.

Fichiers modifies

  • view/digiriskstandard/actionplan_list.php

Issue

#4765

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant