Revert SQL optimization in videos list :(
authorChocobozzz <me@florianbigard.com>
Fri, 27 Jul 2018 10:01:31 +0000 (12:01 +0200)
committerChocobozzz <me@florianbigard.com>
Fri, 27 Jul 2018 10:03:27 +0000 (12:03 +0200)
server/models/video/video.ts

index 3a3cfbe85452a22000452ae440df21ac495604e3..33d62926b094528cec394ec158b3c44f6fc40984 100644 (file)
@@ -215,8 +215,8 @@ type AvailableForListOptions = {
               'INNER JOIN "videoChannel" ON "videoChannel"."id" = "video"."channelId" ' +
               'INNER JOIN "account" ON "account"."id" = "videoChannel"."accountId" ' +
               'INNER JOIN "actor" ON "account"."actorId" = "actor"."id" ' +
-              'WHERE "actor"."serverId" IS NULL OR ' +
-              '"actor"."id" IN (SELECT "targetActorId" FROM "actorFollow" WHERE "actorId" = 1)' + // Subquery for optimization
+              'LEFT JOIN "actorFollow" ON "actorFollow"."targetActorId" = "actor"."id" ' +
+              'WHERE "actor"."serverId" IS NULL OR "actorFollow"."actorId" = ' + actorIdNumber +
             ')'
           )
         },
@@ -632,7 +632,7 @@ export class VideoModel extends Model<VideoModel> {
     // Do not wait video deletion because we could be in a transaction
     Promise.all(tasks)
       .catch(err => {
-        logger.error('Some errors when removing files of video %s in after destroy hook.', instance.uuid, { err })
+        logger.error('Some errors when removing files of video %s in before destroy hook.', instance.uuid, { err })
       })
 
     return undefined