diff options
author | Sijmen Schoon <me@sijmenschoon.nl> | 2021-10-17 20:08:27 +0300 |
---|---|---|
committer | backportbot[bot] <backportbot[bot]@users.noreply.github.com> | 2021-11-01 14:08:31 +0300 |
commit | ec3b1ccd91ed7b29818b79863638a1f5b7302e8a (patch) | |
tree | bb0aec1f300e31659168efd752cc6e5e683000aa | |
parent | 3c1426a477f0336333f29553ece2a6011bab09b5 (diff) |
Limit parameter count per query in Cache.removeChildren
Signed-off-by: Sijmen Schoon <me@sijmenschoon.nl>
-rw-r--r-- | lib/private/Files/Cache/Cache.php | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/lib/private/Files/Cache/Cache.php b/lib/private/Files/Cache/Cache.php index 1c30a5953c6..4572217325a 100644 --- a/lib/private/Files/Cache/Cache.php +++ b/lib/private/Files/Cache/Cache.php @@ -587,10 +587,13 @@ class Cache implements ICache { return $cacheEntry->getId(); }, $children); - $query = $this->getQueryBuilder(); - $query->delete('filecache_extended') - ->where($query->expr()->in('fileid', $query->createNamedParameter($childIds, IQueryBuilder::PARAM_INT_ARRAY))); - $query->execute(); + $childIdChunks = array_chunk($childIds, 2048); + foreach ($childIdChunks as $childIdChunk) { + $query = $this->getQueryBuilder(); + $query->delete('filecache_extended') + ->where($query->expr()->in('fileid', $query->createNamedParameter($childIdChunk, IQueryBuilder::PARAM_INT_ARRAY))); + $query->execute(); + } /** @var ICacheEntry[] $childFolders */ $childFolders = array_filter($children, function ($child) { @@ -602,10 +605,13 @@ class Cache implements ICache { } } - $query = $this->getQueryBuilder(); - $query->delete('filecache') - ->whereParentIn($parentIds); - $query->execute(); + $parentIdChunks = array_chunk($parentIds, 2048); + foreach ($parentIdChunks as $parentIdChunk) { + $query = $this->getQueryBuilder(); + $query->delete('filecache') + ->whereParentIn($parentIdChunk); + $query->execute(); + } } /** |