Link to discussion about Model.save() not being bulk-friendly from bulk_update() caveats

[Lam Phung]

The documentation for QuerySet.update() briefly mentions that the model’s save() method is not called and pre_save/post_save signals are not sent. However, the docs do not explain why this matters or what those signals normally do. This could be confusing for new contributors who are understanding the difference between update(), and manually calling save() on each object. The current documentation could be updated to include information on why selecting the wrong method could disrupt current logic, or when it is safe/unsafe to use update().

[Youngkwang Yang]

(Note: The screenshot you attached is from the bulk_update() section, not the update() section. "QuerySet.update()" is linked, and clicking it takes you to the update() section.)

there is potential for confusion here since the bulk_update() documentation doesn't explain that update() is called internally. This might explain why the bulk_update() section could be mistaken for the update() section.

[Youngkwang Yang]

Additionally, the QuerySet.update() documentation's final paragraph provides more detailed explanation about signals:
(​https://docs.djangoproject.com/en/5.2/ref/models/querysets/#update)

Finally, realize that update() does an update at the SQL level and, thus, does not call any save() methods on your models, nor does it emit the pre_save or post_save signals (which are a consequence of calling Model.save()). If you want to update a bunch of records for a model that has a custom save() method, loop over them and call save(), like this:

for e in Entry.objects.filter(pub_date__year=2010):
    e.comments_on = False
    e.save()

What are your thoughts on this explanation?

[Jacob Walls]

Thanks for the ticket.

I would take a PR that linked to the discussion about overriding the model save() method, which finishes with an admonition about it being not bulk-friendly. (Even better if we could link directly to that admonition, haven't tried doing so in Sphinx.)

See:
​https://github.com/django/django/blob/97acd4d2f92eef8c285bac070d437bf0fd52e071/docs/topics/db/models.txt#L967-L975

[VIZZARD-X]

PR opened: ​https://github.com/django/django/pull/20274

[Jacob Walls <jacobtylerwalls@…>]

In 11f9fae:

Fixed #36741 -- Linked to custom save()/delete() caveats in docs/ref/models/querysets.txt.