Development #21949
Perf de la vue par formulaires
0%
Description
La vue globale existe grâce à l'SQL et devrait être la vue par défaut partout; pour autant la vue par formulaires pourrait quand même être optimisée.
Fichiers
Révisions associées
backoffice: use SQL views to compute counts in management view (#21949)
backoffice: include "empty site" message in classic forms view (#21949)
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0003-backoffice-include-empty-site-message-in-classic-for.patch 0003-backoffice-include-empty-site-message-in-classic-for.patch ajouté
- Fichier 0002-backoffice-use-SQL-views-to-compute-counts-in-manage.patch 0002-backoffice-use-SQL-views-to-compute-counts-in-manage.patch ajouté
- Fichier 0001-backoffice-don-t-retrieve-formdefs-twice-in-manageme.patch 0001-backoffice-don-t-retrieve-formdefs-twice-in-manageme.patch ajouté
- Patch proposed changé de Non à Oui
Pour info, baseline locale, 1,77 secondes. (208 formulaires).
Premier truc sans rapport avec l'SQL, les formdefs étaient chargés deux fois (une fois pour la page en elle-même et une fois pour la barre latérale, pour voir si afficher un lien "vue carte"); ça fait 200ms économisées.
Ensuite l'origine de ce ticket, profiter de la vue SQL pour produire les totaux présents dans les badges, genre 1 seconde gagnée ici.
Finalement sans rapport avec les perfs mais amené par la dépendance à la vue wcs_all_forms (qui n'existe pas s'il n'y a pas de formdef), l'affichage d'un message quand c'est le cas. (message qu'on avait déjà pour la vue globale).
Arrivée à 0,53 secondes.
Mis à jour par Thomas Noël il y a environ 6 ans
Sur 0002, les counts = {} me semblent de trop dans le sql.py
A part ça, ack.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de Nouveau à En cours
Sur 0002, les counts = {} me semblent de trop dans le sql.py
Yep (ça indique que je n'étais pas parti pour faire ce dict comprehension de jeunot…).
~~
Je ne pousse pas, j'étais parti pour avoir ça sur la mise en prod suivante.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit da19af24c8cc027751c98f615aacee76d52bef94
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Feb 18 14:39:18 2018 +0100
backoffice: include "empty site" message in classic forms view (#21949)
commit ba5936593e5b6776eabf8396d65ddaf84c91f7eb
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Feb 18 14:31:38 2018 +0100
backoffice: use SQL views to compute counts in management view (#21949)
commit b0ecbc544feaad18d1d59cca1dfb181612e3aea9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Feb 18 14:30:50 2018 +0100
backoffice: don't retrieve formdefs twice in management view (#21949)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: don't retrieve formdefs twice in management view (#21949)