Move query outside the loop and reduce chunk size to 1000

This involved changing CacheQueryBuilder\whereParentIn to take a
parameter name, renaming the function accordingly.

Signed-off-by: Sijmen Schoon <me@sijmenschoon.nl>

1 files changed, 12 insertions, 10 deletions

diff --git a/lib/private/Files/Cache/Cache.php b/lib/private/Files/Cache/Cache.php
index f471b5c7838..cba2bae6ed9 100644
--- a/lib/private/Files/Cache/Cache.php
+++ b/lib/private/Files/Cache/Cache.php
@@ -587,11 +587,12 @@ class Cache implements ICache {
 				return $cacheEntry->getId();
 			}, $children);
 
-			$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 = $this->getQueryBuilder();
+			$query->delete('filecache_extended')
+				->where($query->expr()->in('fileid', $query->createParameter('childIds')));
+			
+			foreach (array_chunk($childIds, 1000) as $childIdChunk) {
+				$query->setParameter('childIds', $childIdChunk, IQueryBuilder::PARAM_INT_ARRAY);
 				$query->execute();
 			}
 
@@ -605,11 +606,12 @@ class Cache implements ICache {
 			}
 		}
 
-		$parentIdChunks = array_chunk($parentIds, 2048);
-		foreach ($parentIdChunks as $parentIdChunk) {
-			$query = $this->getQueryBuilder();
-			$query->delete('filecache')
-				->whereParentIn($parentIdChunk);
+		$query = $this->getQueryBuilder();
+		$query->delete('filecache')
+			->whereParentInParameter('parentIds');
+
+		foreach (array_chunk($parentIds, 1000) as $parentIdChunk) {
+			$query->setParameter('parentIds', $parentIdChunk, IQueryBuilder::PARAM_INT_ARRAY);
 			$query->execute();
 		}
 	}