From 2db786742503661f7ec0400093508853896a8d95 Mon Sep 17 00:00:00 2001 From: Jakub Kadlcik Date: Jan 13 2018 17:00:11 +0000 Subject: [frontend] use exists instead of all, it is faster --- diff --git a/frontend/coprs_frontend/coprs/logic/builds_logic.py b/frontend/coprs_frontend/coprs/logic/builds_logic.py index 7ac0f58..62d2ffe 100644 --- a/frontend/coprs_frontend/coprs/logic/builds_logic.py +++ b/frontend/coprs_frontend/coprs/logic/builds_logic.py @@ -11,7 +11,7 @@ from sqlalchemy import or_ from sqlalchemy import and_ from sqlalchemy.orm import joinedload from sqlalchemy.orm.exc import NoResultFound -from sqlalchemy.sql import false,true +from sqlalchemy.sql import false, true, exists from werkzeug.utils import secure_filename from sqlalchemy import desc,asc, bindparam, Integer from collections import defaultdict @@ -762,7 +762,7 @@ GROUP BY # then send an action to create module repodata on backend if (build.module and upd_dict.get("status") == StatusEnum("succeeded") - and all(b.status == StatusEnum("succeeded") for b in build.module.builds)): + and not exists(b.status != StatusEnum("succeeded") for b in build.module.builds)): ActionsLogic.send_build_module(build.copr, build.module) for attr in ["results", "built_packages", "srpm_url"]: