#21125 closed Cleanup/optimization (fixed)

Deprecate old URI syntax for caches

Reported by: FunkyBob Owned by: FunkyBob
Component: Core (Cache system) Version: master
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 Changed 20 months ago by FunkyBob

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to FunkyBob
  • Patch needs improvement unset
  • Status changed from new to assigned

comment:2 Changed 20 months ago by timo

  • 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 Changed 20 months ago by aaugustin

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 Changed 20 months ago by FunkyBob

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

comment:5 Changed 20 months ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 4e9f800742c3048402acbaea67ec3c6bc3bd0935:

Fixed #21125 -- Removed support for cache URI syntax

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