celery/django-celery — pull request #293 by jfischer (2013-11-16 20:40:15+00:00)
Title: Potential fix for issue celery/django-celery#289: use manual tranaction mgmt

I see issue celery/django-celery#289 when using the latest release of djcelery and celery with Django 1.4. Perhaps this issue is due to the new autocommit default in Django 1.6?

I assume the intent of the commit calls in on_snapshot() i, to improve performance by committing every N updates rather than for each one. I added the commit_manually decorator to the on_snapshot() method, which (I think) should leave the transaction open until an explicit commit. I always make sure we commit before leaving on_snapshot() if the transaction status is dirty. This keeps commit_manually happy and is probably a good idea anyway.