Merge pull request #1499 from argos-ci/optimize-query

perf: optimize builds query
argos-ci · Jan 2, 2025 · d76d5fb · d76d5fb
2 parents 1e7dc74 + b8425d1
commit d76d5fb
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/apps/backend/src/graphql/definitions/Project.ts b/apps/backend/src/graphql/definitions/Project.ts
@@ -2,7 +2,7 @@ import { assertNever } from "@argos/util/assertNever";
 import { invariant } from "@argos/util/invariant";
 import * as Sentry from "@sentry/node";
 import gqlTag from "graphql-tag";
-import type { PartialModelObject } from "objection";
+import { raw, type PartialModelObject } from "objection";
 
 import {
   Account,
@@ -516,6 +516,7 @@ export const resolvers: IResolvers = {
               if (!status.includes(IBuildStatus.Accepted)) {
                 qb.whereNotExists(
                   ScreenshotDiff.query()
+                    .select(raw("1"))
                     .whereRaw('screenshot_diffs."buildId" = builds.id')
                     .where("validationStatus", "accepted"),
                 );
@@ -524,6 +525,7 @@ export const resolvers: IResolvers = {
               if (!status.includes(IBuildStatus.Rejected)) {
                 qb.whereNotExists(
                   ScreenshotDiff.query()
+                    .select(raw("1"))
                     .whereRaw('screenshot_diffs."buildId" = builds.id')
                     .where("validationStatus", "rejected"),
                 );
@@ -535,6 +537,7 @@ export const resolvers: IResolvers = {
                     .orWhereNull("conclusion")
                     .orWhereExists(
                       ScreenshotDiff.query()
+                        .select(raw("1"))
                         .whereRaw('screenshot_diffs."buildId" = builds.id')
                         .whereIn("validationStatus", ["rejected", "accepted"]),
                     );
@@ -547,6 +550,7 @@ export const resolvers: IResolvers = {
                     .orWhereNull("conclusion")
                     .orWhereExists(
                       ScreenshotDiff.query()
+                        .select(raw("1"))
                         .whereRaw('screenshot_diffs."buildId" = builds.id')
                         .whereIn("validationStatus", ["rejected", "accepted"]),
                     );