Allowed memory size exhausted when using revisions caching

Created on 10 July 2025, about 1 month ago

In 📌 ContentEntityStorageBase::loadRevision() should use the static and the persistent entity cache like ContentEntityStorageBase::load() Needs work there is work being done to add caching for node revisions. When using the patch for the node revision cache, nodes with a lot of revisions can cause out-of-memory errors while running the queue.
I guess this makes sense, some NodeRevisionDelete plugins load all revisions and when these are cached this can add up to a huge amount.

Implement the latest patch in 📌 ContentEntityStorageBase::loadRevision() should use the static and the persistent entity cache like ContentEntityStorageBase::load() Needs work
Enable the node revision delete module
Create a node with a loth of revisions
Try to run the queue and observe out-of-memory errors

Reset the revision cache in plugins that load the revisions.

None

None

None

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇳🇱Netherlands seanB Netherlands

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @seanB
Comment about 1 month ago →
🇳🇱Netherlands seanB Netherlands
Attached patch seems to fix it for us.
Comment 12 days ago →
🇬🇧United Kingdom catch
📌 Add an in-memory LRU cache Needs work is already in the 11.x branch, so once 📌 ContentEntityStorageBase::loadRevision() should use the static and the persistent entity cache like ContentEntityStorageBase::load() Needs work lands it will only be in versions that are using the LRU cache. I think this is probably duplicate of the LRU issue but moving to needs more info since it's not my project to close issues in.

Production build 0.71.5 2024