- Issue created by @mglaman
- Merge request !1417Opt into static_cache for component config entities β (Open) created by mglaman
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
π€ This seems fine, because it's just PHP memory consumption that would grow, not APCu.
I'd prefer to do this first, and to then re-assess whether β¨ PageRegion::loadForTheme can statically regions cache per-theme Active is still relevant.
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
This looks good to me, but I'd love to have @larowlan confirm that he also sees no reason to NOT do this.
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
I don't have any objections to this
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Great β this now just needs updates to 3 tests, which apparently rely on the fact that there has not been static caching so far.
@mglaman: that screenshot in the issue summary, which route is that for, and for how big of a component tree is it?
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Related: π Implement `lookup_keys` in Component for optimized queries Active β this issue is for making (repeated) loading of a specific
Component
(with known ID) faster. #3528847 is about making querying them faster. - πΊπΈUnited States mglaman WI, USA
@mglaman: that screenshot in the issue summary, which route is that for, and for how big of a component tree is it?
It was the GET on experience_builder.api.layout.get. The tree wasn't huge, maybe 12 components with various nested slots.