Почему не работают транзакции в джанго?

Question

[Виктор Паперно]

У меня есть две модели. Для простоты - модель `ModelA` и `ModelB`

class ModelA():
    pass


def create_model_a(params):
    # do_something
    obj = ModelA(params)
    obj.save()

class ModelB():
    def save(self):
        super(ModelB, self).save()
        create_model_a(params={"id":self.id})

Я хочу, что бы если функция `create_model_a` падала, то и объект модели B не создавался. Пытаюсь обернуть в транзакцию - но не работает.

Что я делал:
1) добавлял декоратор `@transaction.atomic` к функции `save`.
2) Пытался юзать как контекстный менеджер

Как тестил:
Написал функцию, которая один раз работает правильно, а на второй запуск рейзит ошибку `django.db.IntegrityError`

Comments

[Виктор Паперно]

Justa Gain, примеры добавил, а вот вынести, к сожалению - так нельзя, по условиям задачи :(

[Виктор Паперно]

Justa Gain, что есть то есть

[Виктор Паперно]

Это требования тимлида руководства...
И юзаем старый джанго 2.2

[Сергей Горностаев]

Виктор Паперно, не повезло вам попасть в команду, в которой самый старший разработчик не умеет программировать. Прямо в документации Django написано, что переопределить save - это плохая практика.

[Модератор]

Не надо ставить как можно больше тэгов. Лучше оставить один, но конкретный, с которым проблема.
См.п.3.1 Регламента.

[Сергей Еремин]

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

На самом деле транзакции очень коварно себя ведут когда заканчивается память (и диск). Они не откатываются ив се виснет. Как такой случай обрабатывать на питоне и джанге -- вообще не понятно. К счастью за много лет -- один раз такое случалось.

[Сергей Еремин]

Сергей Горностаев, Практика нормальная если нужно изменить поведение модели. Запихивать альтернативное поведение в бизнес-логику, а после ещё и в админку -- еще более плохая практика. Но если штатную админку не предполагается использовать -- то согласен, логичнее все вынести в бизнес-логику приложения.

[Сергей Горностаев]

Сергей Еремин, не нормальная. Конкретно создание экземпляра одной модели при сохранении другой правильно делать в обработчиках сигналов.

[Сергей Горностаев]

Justa Gain, создатели фреймворка так не считают.

[Сергей Горностаев]

Justa Gain, у них вполне сознательный выбор в пользу толстых моделей как раз ради отказа от сервисного слоя.

[Сергей Горностаев]

Justa Gain, с этим соглашусь.