Produits Entr'ouvert » Chrono

Valentin Deniaud a écrit :

Il faut aussi les afficher dans la mini liste des exceptions qu'on voit par guichet sur la page de paramétrage d'un agenda.

Oui bien vu, voilà.

Et en faisant ça, on remarque que la propritété from_settings sert exactement aux mêmes endroits que exception.unavailability_calendar, ça vaut peut être le coup de la remplacer par un can_be_deleted qui engloberait les deux.

J'ai opté pour read_only parce que ça détermine aussi l'affichage d'un lien vers la vue d'édition.

Et tout ceci fait augmenter significativement le nombre de requêtes en DB sur la vue settings cf le test modifié.
Pas trivial d'optimiser ça, je me suis perdu un bon moment dans la doc de l'ORM django sans rien en retirer d'utile, je laisserai bien l'optimisation éventuelle pour plus tard ou pour quelqu'un dans un autre ticket sauf si quelqu'un voit tout de suite un truc évident.

Emmanuel Cazenave a écrit :

Et tout ceci fait augmenter significativement le nombre de requêtes en DB sur la vue settings cf le test modifié.
Pas trivial d'optimiser ça, je me suis perdu un bon moment dans la doc de l'ORM django sans rien en retirer d'utile, je laisserai bien l'optimisation éventuelle pour plus tard ou pour quelqu'un dans un autre ticket sauf si quelqu'un voit tout de suite un truc évident.

Je note les choses plus explicitement pour ma compréhension : le truc qui fait exploser les requêtes c'est que timeperiodexception_set.all() était gratuit parce que préfetché, et là le remplacer par un filter fait une requête. Pas dramatique en soit, sauf que le code est écrit avec le prefetch en tête : le queryset gratuit et réévalué quatre fois par guichet, et comme on crée 10 guichets dans le test, bim 40 requêtes en plus.

Je ne sais pas si on s'en sort avec un prefetch compliqué ou si il faut réécrire le python : un autre ticket pour ça c'est bien, mais peut-être remplacer le commentaire qui dit que timeperiodexception_set est préfetché par un FIXME ?

Pour la compréhension oui c'est un truc dans ce goût là, get_exceptions_within_two_weeks utilisait self.timeperiodexception_set qui avait été préfetché, maintenant on ne peux plus se servir self.timeperiodexception_set puisqu'il faut aussi aller chercher des exceptions venant de calendriers d'indispo.

A labourer la doc django plus quelques tentatives je ne crois pas qu'on puisse utilise prefetch_related ou d'autre bidouilles de l'ORM pour ce cas, je crois que le seul chemin c'est de taper des requêtes sur mesure dans get_context_data et de mettre les données en forme pour une exploitation dans le template. Of course je peux me planter.

J'ai mis à jour le commentaire.

commit 2370a8eafa336f96d02087b13ea4357c71254861
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Nov 4 14:16:32 2020 +0100

    manager: grab exceptions from unavailability calendars (#47395)