Use bootstrap cache bin for components

Created on 20 January 2025, 3 months ago

Problem/Motivation

Similar to 📌 Review cache bin usage of ThemeExtensionList and Theme cache context Active , I think it makes sense to use the bootstrap cache bin since this is information that's used on basically every request including dynamic page cache hits.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Active

Version

3.0

Component

Code

Created by

🇨🇭Switzerland berdir Switzerland

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @berdir
  • Merge request !45Use bootstrap cache → (Merged) created by berdir
  • 🇨🇭Switzerland berdir Switzerland

    Created a merge request.

  • Pipeline finished with Success
    3 months ago
    Total: 151s
    #415894
  • Status changed to Needs review 5 days ago
  • Pipeline finished with Skipped
    5 days ago
    #475527
  • 🇦🇺Australia acbramley

    I don't fully grok how different cache bins interact with different parts of the request but I trust your judgement here :) thanks!

  • 🇨🇭Switzerland berdir Switzerland

    Thanks for merging.

    TLDR: bootstrap (and config + discovery) cache bins use APCu to keep the caches in memory. All other cache bins always do a database/redis/memcache query. Things that are used often (this is used on dynamic page cache hits that need to render placeholders), aren't big and don't have many variations benefit from being put in those cache bins.

Production build 0.71.5 2024