Development #23919
Utiliser un afterjob pour la mise à jour des digest
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
18 mai 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
J'ai réutilisé l'infra de réindexation mais ça doit juste être parce qu'elle venait d'arriver et que je l'avais en tête; ça serait mieux de faire ça en afterjob, ça aurait l'avantage de marcher en pickle et sql.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 6 ans
Oui, certainement mieux.
Le cas limite que je vois, mais on peut passer outre, c'est deux re-indexations qui seraient lancées en parallèle suite à deux changement de digest un peu trop rapprochés. (en sql on pourrait gérer un statut reindex_foo = 'in-progress'...). Je pense qu'on peut passer outre en première approche.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier 0001-misc-use-an-afterjob-to-update-form-digests-23919.patch 0001-misc-use-an-afterjob-to-update-form-digests-23919.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Passer outre, oui.
Mis à jour par Thomas Noël il y a presque 6 ans
...select(order_by='id') ou considérer que c'est pas si grave et pousser ainsi.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
Ajouté l'order_by. Et corrigé l'updading en updating.
commit 4dc3d6ea3a57effd539b10b69747f6e2ba7a76de
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri May 18 11:25:33 2018 +0200
misc: use an afterjob to update form digests (#23919)
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
misc: use an afterjob to update form digests (#23919)