#18676: Django should do m2m deletes in a single query when possible
-------------------------------------+-------------------------------------
     Reporter:  Jeremy Dunck         |                    Owner:  nobody
         Type:                       |                   Status:  closed
  Cleanup/optimization               |
    Component:  Database layer       |                  Version:  1.4
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:  fixed
     Keywords:                       |             Triage Stage:  Ready for
                                     |  checkin
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Mariusz Felisiak <felisiak.mariusz@…>):

 In [changeset:"26c4be2ebe918601c0a13d4abb3f3cab49107c73" 26c4be2e]:
 {{{
 #!CommitTicketReference repository=""
 revision="26c4be2ebe918601c0a13d4abb3f3cab49107c73"
 Refs #18676 -- Enabled fast-delete for m2m_changed senders.

 There's no reason to disable fast-delete when an intermediary
 many-to-many model has connected m2m_changed receivers because the
 signal is only sent when related manager's clear() and remove() methods
 are directly called.

 This must have been overlooked in 1cd6e04cd4f768bcd4385b75de433d497d938f82
 given no regression tests fail when m2m_changed is not taken into
 consideration to determine if fast-delete can be enabled.
 }}}

-- 
Ticket URL: <https://code.djangoproject.com/ticket/18676#comment:14>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.5a08fafecfae824487a6365c0a930194%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to