Project

General

Profile

Development #69109

api: performance du endpoint list, charger les parts de manière paresseuse

Added by Benjamin Dauvergne 5 months ago. Updated 3 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
15 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Cf. les résultats cProfile dans #69090

Même sans afficher les évolutions il est nécessaire d'accéder à certains champs de l'objet Evolution, des champs qui ne coûtent pas cher à charger, mais les parts par contre coûtent assez cher à lire à cause de l'utilisation de pickle, il faudrait rendre leur chargement paresseux. C'est un préalable à #69018 je pense.


Files


Related issues

Related to w.c.s. - Development #69090: Requête web beaucoup trop longue sur /api/cardsFermé14 September 2022

Actions
Related to w.c.s. - Development #69108: api: performance du endpoint list, permettre d'exclure les evolutionsFermé15 September 2022

Actions

Associated revisions

Revision cf2c8cb2 (diff)
Added by Benjamin Dauvergne 3 months ago

sql: load Evolution.parts lazily from database (#69109)

History

#2

Updated by Benjamin Dauvergne 5 months ago

#4

Updated by Benjamin Dauvergne 5 months ago

  • Blocks Development #69108: api: performance du endpoint list, permettre d'exclure les evolutions added
#5

Updated by Benjamin Dauvergne 5 months ago

#6

Updated by Benjamin Dauvergne 5 months ago

#7

Updated by Benjamin Dauvergne 5 months ago

  • Description updated (diff)
#8

Updated by Benjamin Dauvergne 5 months ago

  • Assignee set to Benjamin Dauvergne
#9

Updated by Benjamin Dauvergne 5 months ago

#10

Updated by Frédéric Péters 5 months ago

  • Blocks deleted (Development #69108: api: performance du endpoint list, permettre d'exclure les evolutions)
#11

Updated by Frédéric Péters 5 months ago

  • Related to Development #69108: api: performance du endpoint list, permettre d'exclure les evolutions added
#12

Updated by Frédéric Péters 5 months ago

(uniquement des liaisons simples entre tickets, pas de bloque ou autre).

#13

Updated by Frédéric Péters 3 months ago

Comme on n'imagine pas ça pour autre chose que l'évolution, renommer en LazyEvolutionList ?

Ok l'exhaustivité mais il y a des méthodes qui ne sont jamais utilisées; bêtement pour le coverage il y aurait moyen d'ajouter ne fut-ce qu'un mini-test bidon qui les utilise ? (ou les supprimer ?).

#15

Updated by Frédéric Péters 3 months ago

  • Status changed from Solution proposée to Solution validée

Merci ok.

#16

Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit cf2c8cb29385db8bdee00dec384be7598bf3cf17
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Sep 15 12:25:15 2022 +0200

    sql: load Evolution.parts lazily from database (#69109)
#17

Updated by Transition automatique 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#18

Updated by Transition automatique 21 days ago

Automatic expiration

Also available in: Atom PDF