Opened 2 years ago

Last modified 2 years ago

#34135 closed New feature

Add async interface to ManyRelatedManager & RelatedManager — at Initial Version

Reported by: Jon Janzen Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords: async
Cc: Carlton Gibson Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

With #34112 and #33646 combined, there is now an async interface for reading/writing Models in a lot of cases. One of the obvious places where that isn't true is in RelatedManager instances. RelatedManager instances are missing async versions of:

  • add
  • remove
  • clear
  • set

And this might be a separate ticket but the docs [1] are missing a reference to acreate which according to dir() exists on ManyRelatedManager (I assume as a consequence of inheriting from some superclass Manager type) but I didn't actually try and see if it works. I'm suspicious that it might not be fully correct as related_descriptors.py includes a possibly custom version of the synchronous create function (see [2] & [3]).

[1] ​https://docs.djangoproject.com/en/dev/ref/models/relations/#django.db.models.fields.related.RelatedManager
[2] ​https://github.com/django/django/blob/main/django/db/models/fields/related_descriptors.py#L788-L792
[3] ​https://github.com/django/django/blob/main/django/db/models/fields/related_descriptors.py#L1186-L1190

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top