Development #81534
ajouter quelques conn.commit()
0%
Description
Il y a une bascule vers autocommit dans #81133, en l'attendant il y a deux points notés qui pourraient être traités,
Le code de wcs/sql.py est couvert de commit() sur la plupart des chemins de code. Mais il en subsiste qui n'ont pas cet appel, comme SqlMixin::get, get_ids_from_query.
À l'inverse, certaines fonctions enchaînent les .execute/.commit (par exemple SqlUser::get_reference_ids), ce qui est contre-productif puisque cela multiplie sans raison les fermetures de transaction alors que la ligne de code suivante va devoir en rouvrir une.
Demandes liées
Révisions associées
sql: remove excessive calls to commit() in get_reference_ids (#81534)
Historique
Mis à jour par Frédéric Péters il y a 7 mois
- Lié à Development #81133: wcs et transactions PG ajouté
Mis à jour par Robot Gitea il y a 7 mois
- Statut changé de Nouveau à En cours
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/708
- Titre : WIP: ajout de quelques conn.commit() (#81534)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/708/files
Mis à jour par Robot Gitea il y a 7 mois
- Statut changé de Solution proposée à Solution validée
Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 7 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/708
- Titre : ajout de quelques conn.commit() (#81534)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/708/files
Mis à jour par Transition automatique il y a 7 mois
- Statut changé de Résolu (à déployer) à Solution déployée
sql: add missing commit() to a few methods (#81534)