Opened 9 years ago

Closed 2 years ago

Last modified 2 years ago

#25996 closed Cleanup/optimization (fixed)

Revise the Performance section in topics/http/urls

Reported by: Tzu-ping Chung Owned by: Ralph Reid
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: pramodpsb@…, Marten Kenbeek 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

The section, written back in the 0.9 days during 2005, goes:

Each regular expression in a urlpatterns is compiled the first time it’s accessed. This makes the system blazingly fast.

https://docs.djangoproject.com/en/dev/topics/http/urls/#performance

While the first part of the sentence is true, it does not serve as a reason why the dispatcher is blazingly fast. And the general public perception these days is almost the opposite: Django’s URL dispatcher is thought to be between “not fast” and “slow”, depending on what standard you compare it against.

While I am happy with Django’s URL dispatcher, and certainly don’t have anything to complain, it might be a good idea to remove the ten-year-old statement that is not valid anymore.

Change History (12)

comment:1 by Pramod Bisht, 9 years ago

Cc: pramodpsb@… added

comment:2 by Tim Graham, 9 years ago

Cc: Marten Kenbeek added
Easy pickings: unset
Summary: Remove the Performance section in topics/http/urlsRevise the Performance section in topics/http/urls
Triage Stage: UnreviewedAccepted

I'd suggest to revise the section to describe what parts of the URL system are fast (routing URLs, I think) and what parts are slow (reversing URLs, I think). There are probably other considerations. Marten might be in a good position to do this since he's been working on some URL dispatcher changes.

comment:3 by j-shu, 8 years ago

Owner: changed from nobody to j-shu
Status: newassigned

comment:5 by Tim Graham, 8 years ago

Patch needs improvement: set

Comments for improvement are on the PR.

comment:6 by Mariusz Felisiak, 4 years ago

Owner: j-shu removed
Status: assignednew

comment:7 by Ralph Reid, 2 years ago

Owner: set to Ralph Reid
Status: newassigned

comment:9 by Carlton Gibson, 2 years ago

Patch needs improvement: unset

comment:10 by Mariusz Felisiak, 2 years ago

Triage Stage: AcceptedReady for checkin

comment:11 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

Resolution: fixed
Status: assignedclosed

In d3579ca1:

Fixed #25996 -- Revised performance section in URLs topic docs.

comment:12 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

In 950a84ce:

[4.1.x] Fixed #25996 -- Revised performance section in URLs topic docs.

Backport of d3579ca1124a163cad760558e8d138984e7a10b4 from main

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