Projet

Général

Profil

Development #22107

Ne pas recalculer les variables de substitution inutilement

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
25 février 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Le calcul des variables de substitution prend un temps non négligeable et est joué plusieurs fois lors du traitement d'une requête (par exemple une fois par action d'envoi de mail, ou par action "alerte", etc.), pour réduire ça, on peut conserver le résultat le long d'une requête (tant que les données ne changent pas).

(#22106 est une autre approche, à plus long terme)


Fichiers

Révisions associées

Révision 95651fc8 (diff)
Ajouté par Frédéric Péters il y a environ 6 ans

general: cache substitution variables in requests (#22107)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 6 ans

#2

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

J'ai à peu près réussi à suivre sans rien connaitre à wcs, donc ça doit être pas mal.
Du coup petit ack (car petit niveau de compréhension de mon coté).

#3

Mis à jour par Thomas Noël il y a environ 6 ans

Il y a une raison à ne pas utiliser le décorateur @invalidate_cache sur Substitutions.reset et feed ?

Par ailleurs, bien qu'il y ait le "from Substitutions import ...", je verrais bien ce décorateur renommé @invalidate_substitions_cache (ça rendra plus lisible son utilisation, selon moi)

#4

Mis à jour par Frédéric Péters il y a environ 6 ans

Il y a une raison à ne pas utiliser le décorateur @invalidate_cache sur Substitutions.reset et feed ?

Je pense c'est juste qu'au début du patch je n'avais pas créé de décorateur; c'est modifié.

Par ailleurs, bien qu'il y ait le "from Substitutions import ...", je verrais bien ce décorateur renommé @invalidate_substitions_cache (ça rendra plus lisible son utilisation, selon moi)

Renommé ainsi, ok.

#5

Mis à jour par Thomas Noël il y a environ 6 ans

Ack

#6

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 95651fc809b0d52e1997530c4a4748a5720a38e4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Feb 25 14:14:27 2018 +0100

    general: cache substitution variables in requests (#22107)
#7

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

Formats disponibles : Atom PDF