Problem/Motivation

Similar to #3500687: Use bootstrap cache bin in ThemeExtensionList, 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

Issue fork components-3500811

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

berdir created an issue. See original summary.

berdir opened merge request !45

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review

Created a merge request.

acbramley’s picture

acbramley commented

  • acbramley committed 182c4198 on 3.x authored by berdir 
    Issue #3500811 by berdir: Use bootstrap cache bin for components
acbramley’s picture

acbramley commented
Status: Needs review » Fixed

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

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

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.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.