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.
Related issues
Associated revisions
sql: remove excessive calls to commit() in get_reference_ids (#81534)
History
Updated by Frédéric Péters 12 months ago
- Related to Development #81133: wcs et transactions PG added
Updated by Robot Gitea 12 months ago
- Status changed from Nouveau to 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
Updated by Robot Gitea 12 months ago
- Status changed from Solution proposée to Solution validée
Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea 12 months ago
- Status changed from Solution validée to 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
Updated by Transition automatique 12 months ago
- Status changed from Résolu (à déployer) to Solution déployée
sql: add missing commit() to a few methods (#81534)