Opened 10 years ago

Closed 9 years ago

#3445 closed (wontfix)

all caching-related code should be collected into its own cache application

Reported by: Gary Wilson <gary.wilson@…> Owned by: nobody
Component: Core (Other) Version: master
Severity: Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

There are various modules related to caching spread throughout the django tree. They should be collected into a "cache" package/application.

Attachments (1)

3445.diff (10.9 KB) - added by Gary Wilson <gary.wilson@…> 10 years ago.
Moved cache related modules into a cache package

Download all attachments as: .zip

Change History (8)

comment:1 Changed 10 years ago by Gary Wilson <gary.wilson@…>

Triage Stage: UnreviewedDesign decision needed

comment:2 Changed 10 years ago by Malcolm Tredinnick

Gary, can you provide a few more details on where the scattering is? It sounds like you already know the details. Not easy to make a decision on this without more information.

comment:3 Changed 10 years ago by Gary Wilson <gary.wilson@…>

./django/core/cache/
./django/views/decorators/cache.py
./django/utils/cache.py
./django/middleware/cache.py

comment:4 Changed 10 years ago by Gary Wilson <gary.wilson@…>

The Object-level caching Summer of Code project makes an even stronger case for a cache package. The author has proposed adding a caching app to contrib, but I suggest we put everything into one cache app.

Changed 10 years ago by Gary Wilson <gary.wilson@…>

Attachment: 3445.diff added

Moved cache related modules into a cache package

comment:5 Changed 10 years ago by Gary Wilson <gary.wilson@…>

Has patch: set

Renames:

  django/core/cache => django/cache
  django/middleware/cache.py => django/cache/middleware.py
  django/utils/cache.py => django/cache/utils.py
  django/views/decorators/cache.py => django/cache/decorators.py

comment:6 Changed 9 years ago by Jacob

I agree that the four different "*.cache" modules are silly, but this'll break a lot of user code just for a bit of cleanup. So here's the $1 million question: is the code churn and renaming "worth" the cleanup?

comment:7 Changed 9 years ago by Jacob

Resolution: wontfix
Status: newclosed

After talking with Malcolm, it just doesn't see like this is worth the breakage.

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