#21688 closed Cleanup/optimization (fixed)

Invalidate internal caches when INSTALLED_APPS changes

Reported by: aaugustin Owned by: nobody
Component: Testing framework Version: master
Severity: Normal Keywords: app-loading
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

We should review all calls to apps.get_apps_configs and check if the result gets cached, either by the caller or one of its callers, recursively.

When results are cached, the cache should be invalidated when INSTALLED_APPS changes.

(Filed under "testing" because that's the only situation where it's legal to change settings and the changes go in django.test.signals.)

Change History (6)

comment:1 Changed 20 months ago by aaugustin

  • Keywords app-loading added

comment:2 Changed 20 months ago by aaugustin

  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 20 months ago by aaugustin

Obsolete utilities such as TransRealMixin should also be removed as part of this ticket.

comment:4 Changed 20 months ago by Aymeric Augustin <aymeric.augustin@…>

In f17d00278ee4a22421dc8148588d94fe7fa17427:

Wiped get_commands() cache when INSTALLED_APPS changes.

Refs #21018, #21688.

comment:5 Changed 19 months ago by aaugustin

  • Has patch set

PR: https://github.com/django/django/pull/2217

This doesn't have much to do with app-loading, in fact...

comment:6 Changed 19 months ago by Aymeric Augustin <aymeric.augustin@…>

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.
Back to Top