#17: Metasystem optimization: Share select_related in memory
----------------------+-----------------------------------------------------
Reporter: adrian | Owner: adrian
Status: new | Component: Database wrapper
Version: | Resolution:
Keywords: | Stage: Accepted
Has_patch: 1 | Needs_docs: 0
Needs_tests: 1 | Needs_better_patch: 0
----------------------+-----------------------------------------------------
Changes (by [EMAIL PROTECTED]):
* needs_better_patch: 1 => 0
Comment:
Replying to [comment:15 Brian Rosner <[EMAIL PROTECTED]>]:
> Oh and I wanted to mention that I removed some changes in areas that
really don't pertain to what this ticket is trying to accomplish. While
the changes were better they are not really related.
Only unrelated changes (patch wise) you removed were conversion of range
in delete to xrange; those might as well stay (already tweaking that area
of the file).
The tests changes that were dropped (forced conversion of a few test args
up front) however *are* required- run the tests, it'll hork. They're
needed- that's more a sign that the misc fields are screwed up since
they're not forcing unicode conversion, but that's a seperate mess only
exposed by tests.
Either way, a version of this is maintained in a bzr branch at
http://pkgcore.org/~ferringb/bzr/instance-caching/ ; using it for curse
these days, so I keep it reasonably within trunk (week lag is usually the
most).
Related to that, please don't do misc var renames when correcting for bit
rot- not trying to be possessive, moreso it's extra noise when trying to
nail down exactly what was updated between patch versions, and changes the
api the functionality provides when kwargs are mangled.
Finally, dropping the "patch needs improvement"; not really relevant at
this point (my earlier comments have long since been fixed).
--
Ticket URL: <http://code.djangoproject.com/ticket/17#comment:16>
Django Code <http://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 [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
