aboutsummaryrefslogtreecommitdiff
path: root/guix-build-coordinator
diff options
context:
space:
mode:
authorChristopher Baines <mail@cbaines.net>2025-02-09 11:14:27 +0000
committerChristopher Baines <mail@cbaines.net>2025-02-09 11:14:27 +0000
commitcb6a5364af20bf37dcd44a6b10f975ec718659d9 (patch)
tree967a9822d65e80d281c0e47593c704837290254c /guix-build-coordinator
parent4050f45d413029c9851a0c4b38c5d69f771c4345 (diff)
downloadbuild-coordinator-cb6a5364af20bf37dcd44a6b10f975ec718659d9.tar
build-coordinator-cb6a5364af20bf37dcd44a6b10f975ec718659d9.tar.gz
Attempt to speed up datastore-update-unprocessed-builds-for-build-success
Remove the join on unprocessed_builds_with_derived_priorities since this shouldn't actually change the query and we already have the build_id.
Diffstat (limited to 'guix-build-coordinator')
-rw-r--r--guix-build-coordinator/datastore/sqlite.scm12
1 files changed, 5 insertions, 7 deletions
diff --git a/guix-build-coordinator/datastore/sqlite.scm b/guix-build-coordinator/datastore/sqlite.scm
index 3233938..73c6671 100644
--- a/guix-build-coordinator/datastore/sqlite.scm
+++ b/guix-build-coordinator/datastore/sqlite.scm
@@ -1923,7 +1923,7 @@ LIMIT 1"
(sqlite-prepare
db
"
-SELECT DISTINCT unprocessed_builds.id
+SELECT input_builds.id
FROM builds
INNER JOIN derivation_outputs
ON builds.derivation_id = derivation_outputs.derivation_id
@@ -1931,12 +1931,10 @@ INNER JOIN derivation_outputs AS all_derivation_outputs
ON all_derivation_outputs.output_id = derivation_outputs.output_id
INNER JOIN derivation_inputs
ON derivation_inputs.derivation_output_id = all_derivation_outputs.id
-INNER JOIN builds AS unprocessed_builds
- ON unprocessed_builds.processed = 0
- AND unprocessed_builds.derivation_id = derivation_inputs.derivation_id
-INNER JOIN unprocessed_builds_with_derived_priorities
- ON unprocessed_builds_with_derived_priorities.build_id = unprocessed_builds.id
- AND unprocessed_builds_with_derived_priorities.all_inputs_built = 0
+INNER JOIN builds AS input_builds
+ ON input_builds.processed = 0
+ AND input_builds.canceled = 0
+ AND input_builds.derivation_id = derivation_inputs.derivation_id
WHERE builds.id = :build_id"
#:cache? #t)))