Opened 11 years ago

Closed 11 years ago

#21125 closed Cleanup/optimization (fixed)

Deprecate old URI syntax for caches

Reported by: FunkyBob Owned by: FunkyBob
Component: Core (Cache system) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Given it was removed from the docs 3 years ago, when multi-cache support was added:

https://github.com/django/django/commit/673e6fc7fb243ed44841b9969d26a161c25733b3#diff-a7f182984504962fb780894ba7d4746e

it's about time to remove it.

This cleanup will also help the work to add a thread-safe, non-duplicating cache API.

PR - https://github.com/django/django/pull/1642

Change History (5)

comment:1 by FunkyBob, 11 years ago

Owner: changed from nobody to FunkyBob
Status: newassigned

comment:2 by Tim Graham, 11 years ago

Easy pickings: unset

It seems likely to me that people migrated to using the new syntax as part of the changes from the CACHE_BACKEND to CACHES setting and thus a justification for making this change without a deprecation. Objections?

comment:3 by Aymeric Augustin, 11 years ago

Yes, since the old syntax hasn't been supported for years, and since it's very obvious and easy to fix for people still using it, we can remove it without going through a deprecation path.

I would mention it as a backward-incompatible change in the release notes. "The URL syntax still worked, although that wasn't documented nor officially supported. If you're still using it..."

comment:4 by FunkyBob, 11 years ago

I've updated the docs.. please comment on working, etc.

comment:5 by Tim Graham <timograham@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In 4e9f800742c3048402acbaea67ec3c6bc3bd0935:

Fixed #21125 -- Removed support for cache URI syntax

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