Projet

Général

Profil

Bug #6286

Utiliser .rollback() en cas d'erreur à la création de l'index sql

Ajouté par Frédéric Péters il y a plus de 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 janvier 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment il y a juste attrapage de l'erreur mais la connexion postgresql reste en mode "cassé" ("current transaction is aborted, commands ignored until end of transaction block"), un .rollback() ramènerait une situation convenable.


Fichiers

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 9 ans

C'est une erreur qu'on avait eue lors du passage en SQL à l'AUF.

#2

Mis à jour par Thomas Noël il y a plus de 9 ans

C'est vraiment utile ? Selon la doc, un close() est un rollback implicite. Non ?
http://initd.org/psycopg/docs/usage.html#transactions-control

#3

Mis à jour par Frédéric Péters il y a plus de 9 ans

Je préfère être explicite au niveau du rollback, surtout parce qu'ensuite ça permettrait de reprendre le curseur pour d'autres opérations (genre la création d'un autre index, dans ce cas précis).

#4

Mis à jour par Thomas Noël il y a plus de 9 ans

Tu as raison : ack

#5

Mis à jour par Frédéric Péters il y a plus de 9 ans

  • Statut changé de En cours à Résolu (à déployer)
commit b04822353aecee429fcfac5d00d92bf82ac4c419
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jan 12 21:19:04 2015 +0100

    sql: rollback if users_name_idx already exists (#6286)
#6

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF