fix(server): fix inconsistent explore queries (immich-app#5774)

* remove limits * update sql
etnoy · Dec 17, 2023 · e3fa32a · e3fa32a
1 parent 08f66c2
commit e3fa32a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/server/src/infra/repositories/asset.repository.ts b/server/src/infra/repositories/asset.repository.ts
@@ -706,9 +706,7 @@ export class AssetRepository implements IAssetRepository {
       .createQueryBuilder('e')
       .select('city')
       .groupBy('city')
-      .having('count(city) >= :minAssetsPerField', { minAssetsPerField })
-      .orderBy('random()')
-      .limit(maxFields);
+      .having('count(city) >= :minAssetsPerField', { minAssetsPerField });
 
     const items = await this.getBuilder({
       userIds: [ownerId],
@@ -737,9 +735,7 @@ export class AssetRepository implements IAssetRepository {
       .createQueryBuilder('si')
       .select('unnest(tags)', 'tag')
       .groupBy('tag')
-      .having('count(*) >= :minAssetsPerField', { minAssetsPerField })
-      .orderBy('random()')
-      .limit(maxFields);
+      .having('count(*) >= :minAssetsPerField', { minAssetsPerField });
 
     const items = await this.getBuilder({
       userIds: [ownerId],

diff --git a/server/src/infra/sql/asset.repository.sql b/server/src/infra/sql/asset.repository.sql
@@ -618,10 +618,6 @@ WITH
       city
     HAVING
       count(city) >= $1
-    ORDER BY
-      random() ASC
-    LIMIT
-      12
   )
 SELECT DISTINCT
   ON (c.city) "asset"."id" AS "data",
@@ -653,10 +649,6 @@ WITH
       tag
     HAVING
       count(*) >= $1
-    ORDER BY
-      random() ASC
-    LIMIT
-      12
   )
 SELECT DISTINCT
   ON (unnest("si"."tags")) "asset"."id" AS "data",