Блог

У меня есть функция просмотра:

 @transaction.commit_manually
def xyz(request):
    if ABC:
        success = something()

        if success:
            status = "success"
            transaction.commit()

        else:
            status = "dataerrors"
            transaction.rollback()
    else:
        status = "uploadproblem"
        transaction.rollback()

    return render(request, "template.html", {
        'status': status,
    })
  

Я считаю, что каждый путь кода так или иначе завершает транзакцию. Но Django, похоже, жалуется, что это не так. Есть идеи?

 Django Version:     1.3
Exception Type:     TransactionManagementError
Exception Value:    Transaction managed block ended with pending COMMIT/ROLLBACK
  

РЕДАКТИРОВАТЬ: никакие другие исключения не генерируются для изменения пути кода.

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

По какой-то причине декоратор @transaction.commit_manually отключает исключения, возникающие в функции.

Временно удалите декоратор из вашей функции, теперь вы увидите исключение, исправьте его и верните декоратор обратно!

У меня была та же проблема. Единственным решением, которое я нашел, было использовать предложение try / finally, чтобы гарантировать, что фиксация произойдет после рендеринга.

 @transaction.commit_manually
def xyz(request):
    committed = False
    try:
        if ABC:
            success = something()

            if success:
                status = "success"
                transaction.commit()
                committed = True

            else:
                status = "dataerrors"
                transaction.rollback()
                committed = True
        else:
            status = "uploadproblem"
            transaction.rollback()
            committed = True

        return render(request, "template.html", {
            'status': status,
        })
    finally:
        if not committed:
            transaction.rollback() # or .commit() depending on your error-handling logic
  

Не имеет смысла, но у меня это сработало.

У меня была такая же проблема, и я узнал, что даже если вы правильно закроете транзакцию вручную в случае исключений, если вы затем снова выполните запись в orm в области ручной транзакции, кажется, что транзакция каким-то образом повторно открывается и вызывает исключение транзакции.

             with transaction.commit_manually():
                try:
                    <exciting stuff>
                    transaction.commit()                        
                except Exception, e:
                    transaction.rollback()
                    o.error='failed' <== caused transaction exception
  

еще одна причина, по которой вы можете видеть эту проблему, заключается в том, что у вас в системе несколько баз данных.

Я смог устранить эту ошибку с помощью

 @transaction.commit_manually(using='my_other_db')
def foo():
   try:
        <db query>
        transaction.commit(using='my_other_db')
   except:
        transaction.rollback(using='my_other_db')
  

Это всегда происходит, когда где-то в коде возникает необработанное исключение. В моем случае по какой-то причине исключение не было передано отладчику, что и вызвало у меня путаницу.

У меня была похожая проблема, возможно, этот код отлично подходит для вас:

 @transaction.commit_on_success
def xyz(request):
    if ABC:
        success = something()

        if success:
            status = "success"

        else:
            status = "dataerrors"
            transaction.rollback()
    else:
        status = "uploadproblem"
        transaction.rollback()

    return render(request, "template.html", {
        'status': status,
    })
  

Как говорили другие люди, исключения, возникающие внутри оформленной функции, «теряются», потому что они перезаписываются TransactionManagementError исключением.

Я предлагаю расширить transaction.commit_manually декоратор. Мой декоратор transaction_commit_manually использует transaction.commit_manually декоратор внутренне; если в оформленной функции возникает исключение, мой декоратор перехватывает исключение, выполняет transaction.rollback() и снова создает исключение. Таким образом, транзакция очищена правильно, и исходное исключение не потеряно.

 def _create_decorator_transaction_commit_manually(using=None):
    def deco(f):
        def g(*args, **kwargs):
            try:
                out = f(*args, **kwargs)
            except Exception as e:
                if using is not None:
                    transaction.rollback(using=using)
                else:
                    transaction.rollback()
                raise e
            return out
        if using is not None:
            return transaction.commit_manually(using=using)(g)
        return transaction.commit_manually(g)
    return deco

def transaction_commit_manually(*args, **kwargs):
    """
    Improved transaction.commit_manually that does not hide exceptions.

    If an exception occurs, rollback work and raise exception again
    """
    # If 'using' keyword is provided, return a decorator
    if 'using' in kwargs:
        return _create_decorator_transaction_commit_manually(using=kwargs['using'])
    # If 'using' keyword is not provided, act as a decorator:
    # first argument is function to be decorated; return modified function
    f = args[0]
    deco = _create_decorator_transaction_commit_manually()
    return deco(f)
  

Поместите свой код в блок try / except .В блоке except просто выполните transaction.rollback и зарегистрируйте объект исключения.

 @transaction.commit_manually
def xyz(xyz):
   try:
       some_logic
       transaction.commit()
    except Exception,e:
       transaction.rollback()
       print str(e)
  

У меня была такая же проблема, и я пробовал различные подходы. Вот что сработало для меня, но я не уверен, что это правильный способ сделать это. Измените свой оператор return на:

 with transaction.commit_on_success():
    return render(request, "template.html", {
        'status': status,
    })
  

Профессионалы Django, правильный ли это подход?

При необходимости сделайте резервную копию базы данных и удалите таблицу ROLLBACK_TEST из своей базы данных.

 mysql> DROP TABLE `ROLLBACK_TEST`;