Skip to content

Commit dde8412

Browse files
authored
Merge pull request #53134 from nextcloud/backport/53120/stable30
[stable30] perf: improve query to determine which users need a background scan
2 parents e4261a6 + e089474 commit dde8412

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

apps/files/lib/BackgroundJob/ScanFiles.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ private function getUserToScan() {
7979
$query->select('m.user_id')
8080
->from('filecache', 'f')
8181
->leftJoin('f', 'mounts', 'm', $query->expr()->eq('m.storage_id', 'f.storage'))
82-
->where($query->expr()->lt('f.size', $query->createNamedParameter(0, IQueryBuilder::PARAM_INT)))
82+
->where($query->expr()->eq('f.size', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
8383
->andWhere($query->expr()->gt('f.parent', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
8484
->setMaxResults(10)
8585
->groupBy('f.storage')
@@ -100,7 +100,7 @@ private function getUserToScan() {
100100
$query->select('m.user_id')
101101
->from('filecache', 'f')
102102
->leftJoin('f', 'mounts', 'm', $query->expr()->eq('m.storage_id', 'f.storage'))
103-
->where($query->expr()->lt('f.size', $query->createNamedParameter(0, IQueryBuilder::PARAM_INT)))
103+
->where($query->expr()->eq('f.size', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
104104
->andWhere($query->expr()->gt('f.parent', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
105105
->andWhere($query->expr()->in('f.storage', $query->createNamedParameter($storages, IQueryBuilder::PARAM_INT_ARRAY)))
106106
->setMaxResults(1)
@@ -111,7 +111,7 @@ private function getUserToScan() {
111111
$query->select('m.user_id')
112112
->from('filecache', 'f')
113113
->innerJoin('f', 'mounts', 'm', $query->expr()->eq('m.storage_id', 'f.storage'))
114-
->where($query->expr()->lt('f.size', $query->createNamedParameter(0, IQueryBuilder::PARAM_INT)))
114+
->where($query->expr()->eq('f.size', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
115115
->andWhere($query->expr()->gt('f.parent', $query->createNamedParameter(-1, IQueryBuilder::PARAM_INT)))
116116
->setMaxResults(1)
117117
->runAcrossAllShards();

0 commit comments

Comments
 (0)