#26207 closed Cleanup/optimization (fixed)
Replace dynamic classes with non-data descriptors for deferred instance loading
Reported by: | Anssi Kääriäinen | Owned by: | nobody |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Instead of creating subclasses of the real model with descriptors for the deferred fields, always add non-data descriptors for all concrete fields. Gone is dynamic class creation for deferred loading!
Change History (8)
comment:1 by , 9 years ago
Has patch: | set |
---|---|
Patch needs improvement: | set |
Summary: | New implementation for deferred instance loading → Replace dynamic classes with non-data descriptors for deferred instance loading |
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 9 years ago
Patch needs improvement: | unset |
---|
Updated PR with cosmetic edits + docs. One TODO remains from Anssi's original PR that I'm not sure how to handle.
Note:
See TracTickets
for help on using tickets.
PR and django-dev discussion