#14200: new RegexURLResolver is contructed for every request
-------------------------------------+--------------------------------------
Reporter: Suor | Owner: Suor
Status: new | Milestone:
Component: Core framework | Version: 1.2
Resolution: | Keywords: performance, urlresolvers
Stage: Unreviewed | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 1 |
-------------------------------------+--------------------------------------
Changes (by mk):
* needs_better_patch: => 1
* needs_tests: => 0
* needs_docs: => 0
Comment:
Did you do some performance testing? I'm not sure whether this really
impacts performance in a noticeable negative way.
Anyway, applying this patch breaks the testsuite (FAIL:
test_urlconf_overridden_with_null
(regressiontests.urlpatterns_reverse.tests.RequestURLconfTests)) and
breaks assigning request.urlconf:
http://docs.djangoproject.com/en/dev/topics/http/urls/#how-django-
processes-a-request
The patch at least needs improvement and probably additional justification
too.
--
Ticket URL: <http://code.djangoproject.com/ticket/14200#comment:1>
Django <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.
