#16818: Regression introduced by r16739 -- `ManyRelatedManager.add()` doesn't
commit to database
-------------------------------------+-------------------------------------
     Reporter:  pressureman          |                    Owner:  kmtracey
         Type:  Bug                  |                   Status:  new
    Component:  Database layer       |                  Version:  SVN
  (models, ORM)                      |               Resolution:
     Severity:  Release blocker      |             Triage Stage:  Accepted
     Keywords:                       |      Needs documentation:  0
    Has patch:  0                    |  Patch needs improvement:  0
  Needs tests:  0                    |                    UI/UX:  0
Easy pickings:  0                    |
-------------------------------------+-------------------------------------

Comment (by anonymous):

 The `.add()` issues a BEGIN but no COMMIT. The ROLLBACK will undo the
 uncommitted bulk insert, confirming the bug (missing COMMIT). When fixed,
 the bulk insert will already be committed and the ROLLBACK won't undo it.
 That's my understanding, anyway :) This type of test was suggested to
 ptone by Alex Gaynor on IRC.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/16818#comment:16>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to