Development #47343
Ne plus nettoyer le cache des ContentType entre chaque requête
0%
Description
Ce n'est plus nécessaire depuis que le cache est remplacé par un cache multitenant / thread safe dans django-tenant-schemas.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-multitenant-dont-clear-the-contenttype-cache-between.patch 0001-multitenant-dont-clear-the-contenttype-cache-between.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Comme je me doute qu'il y aura des réticences pour un grand saut dans le vide, j'ai mis ça sous le contrôle d'un setting par défaut à True, on pourra le passer à False et si pas de souci pendant un moment retirer complètement la ligne.
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Moi je dis que tu as l'air de savoir de quoi tu parles (#32248).
Mis à jour par Thomas Noël il y a plus de 3 ans
Je pense aussi que c'est bon, mais ma nature inquiète préférerait qu'on push ça sur le main vendredi prochain seulement histoire que ça tourne bien en recette.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Thomas Noël a écrit :
Je pense aussi que c'est bon, mais ma nature inquiète préférerait qu'on push ça sur le main vendredi prochain seulement histoire que ça tourne bien en recette.
Ce qui ont lu le patch et mon dernier commentaire savent que le patch ne désactive pas le nettoyage du cache, il faut un setting pour cela. Donc vraiment pas d'inquiétude, on pourra ne l'activer que sur la recette, le pousser n'aura aucune conséquence.
Mis à jour par Thomas Noël il y a plus de 3 ans
Benjamin Dauvergne a écrit :
Thomas Noël a écrit :
Je pense aussi que c'est bon, mais ma nature inquiète préférerait qu'on push ça sur le main vendredi prochain seulement histoire que ça tourne bien en recette.
Ce qui ont lu le patch et mon dernier commentaire savent que le patch ne désactive pas le nettoyage du cache, il faut un setting pour cela. Donc vraiment pas d'inquiétude, on pourra ne l'activer que sur la recette, le pousser n'aura aucune conséquence.
Patch mal lu, effectivement (on s'habitue tellement aux doubles négations que quand y'en a pas, on est perdu).
Push it !
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7d0a3162f089eea41bd700029516dac7ab4cc31a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Mon Oct 5 14:28:02 2020 +0200
multitenant: dont clear the contenttype cache between requests (#47343)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
multitenant: dont clear the contenttype cache between requests (#47343)