Какие ошибки в бд при коммите должны в принципе приводить к rollback?

Question

[LakeForest]

Не все ли ошибки в бд при коммите должны в принципе приводить к rollback, или ошибаюсь?
В целом же будет верно обернуть коммиты в try-except на отлов SQLAlchemyError?

Ошибки с коммитами, конечно, возникали только на этапе тестирования/обновления сервиса, но тем не менее. Вывести ошибку в сообщений или логе - мне кажется более правильней, чем ложить весь инстанс.

Answer 1

[Михаил]

Это определяется бизнес-логикой приложения.

Например, если у вас возникает ошибка дублирования уникального ключа - нужно ли откатывать всю транзакцию? Или приложение сможет восстановиться и продолжить работу?

Обернуть потенциально чреватый ошибками код в try-catch - всегда хорошая идея, если у вас есть план как восстанавливаться после ошибки (записать в лог/отправить уведомление/вывести ошибку и продолжить работу - вполне адекватное поведение)

Comments

[LakeForest]

Подскажите, а норм такое поведение?
Откатиться и выбросить исключение. Тогда первая неудачная транзакция не должна блокировать все остальные?

try:
        db_session.commit()
        db_session.refresh(db_result)
    except SQLAlchemyError as e:
        logging.error(e)
        db_session.rollback()
        raise e

[Михаил]

LakeForest, я не в курсе что будет дальше, после выбрасывания исключения. Поэтому ответить не могу.