Development #23180
sql : changer le get_with_indexed_value pour être un générateur
0%
Description
cf #23174
je serais plus à l'aise de transformer get_with_indexed_value en itérateur dans la foulée, oui.
Fichiers
Demandes liées
Révisions associées
storage: change get_with_indexed_value into a generator (#23180)
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
(Passer sur les utilisations de cette méthode m'a fait noter #23179).
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-sql-change-get_with_indexed_value-into-a-generator-2.patch 0001-sql-change-get_with_indexed_value-into-a-generator-2.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 6 ans
- Lié à Bug #23174: optimiser le calcul des demandes expirées ajouté
Mis à jour par Thomas Noël il y a environ 6 ans
Selon moi, le "any", entre autres, ne va pas aller au bout du générateur, et donc n'ira pas jusqu'à conn.commit()/conn.close().
Je ne suis pas assez à l'aise avec les générateurs pour savoir comment gérer cela.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Patch proposed changé de Oui à Non
Tout à fait il faut passer par un context manager (ou oublier le any mais ça serait dommage).
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-sql-change-get_with_indexed_value-into-a-generator-2.patch 0001-sql-change-get_with_indexed_value-into-a-generator-2.patch ajouté
- Patch proposed changé de Non à Oui
Ou try/finally qui est parfait pour ça.
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-storage-change-get_with_indexed_value-into-a-generat.patch 0001-storage-change-get_with_indexed_value-into-a-generat.patch ajouté
Ack.
Est-ce que le get_with_indexed_value de storage.py ne devrait pas également être un générateur, histoire d'être "cohérent" (ça pourrait aussi apporter de l'optimisation, à un moment, même si c'est marginal à l'usage, la bonne vraie optimisation étant de passer en SQL) ?
Au cas où ça ne soit pas si bête, rustine attachée.
Mis à jour par Frédéric Péters il y a environ 6 ans
Comme tu veux, tu peux pousser l'un ou les deux.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
Je viens d'envoyer les deux.
commit 4e596155ac9083e454027abd944265391fe764a0 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Mon Apr 16 16:09:17 2018 +0200 storage: change get_with_indexed_value into a generator (#23180) commit 8c968c4685afc9ef28842090627dd16f11b192a8 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Apr 13 16:07:02 2018 +0200 sql: change get_with_indexed_value into a generator (#23180)
Mis à jour par Thomas Noël il y a presque 6 ans
- Lié à Bug #23551: crash sur l'affichage des paiements ajouté
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
sql: change get_with_indexed_value into a generator (#23180)