From 5af5a6c6aa4c2b31e63d64a43ab647bd6def3808 Mon Sep 17 00:00:00 2001
From: Karen Arutyunov <karen@codesynthesis.com>
Date: Sat, 8 Jul 2023 12:05:09 +0300
Subject: Optimize build-task handler by using object loading view

---
 mod/mod-build-task.cxx | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

(limited to 'mod/mod-build-task.cxx')

diff --git a/mod/mod-build-task.cxx b/mod/mod-build-task.cxx
index a2e1f56..b1f7658 100644
--- a/mod/mod-build-task.cxx
+++ b/mod/mod-build-task.cxx
@@ -1027,7 +1027,9 @@ handle (request& rq, response& rs)
         //
         for (auto& bp: packages)
         {
-          id = move (bp.id);
+          shared_ptr<build_package>& p (bp.package);
+
+          id = p->id;
 
           // If we are in the random package ordering mode, then cache the
           // tenant the start offset refers to, if not cached yet, and check
@@ -1130,8 +1132,7 @@ handle (request& rq, response& rs)
             pkg_query pq (pkg_query::build_tenant::id == id.tenant);
             for (auto& tp: build_db_->query<buildable_package> (pq))
             {
-              shared_ptr<build_package> p (
-                build_db_->load<build_package> (tp.id));
+              shared_ptr<build_package>& p (tp.package);
 
               build_db_->load (*p, p->constraints_section);
 
@@ -1176,7 +1177,7 @@ handle (request& rq, response& rs)
                   b = make_shared<build> (
                     move (bid.package.tenant),
                     move (bid.package.name),
-                    bp.version,
+                    p->version,
                     move (bid.target),
                     move (bid.target_config_name),
                     move (bid.package_config_name),
@@ -1219,8 +1220,6 @@ handle (request& rq, response& rs)
             }
           }
 
-          shared_ptr<build_package> p (build_db_->load<build_package> (id));
-
           for (build_package_config& pc: p->configs)
           {
             pkg_config_name = pc.name;
@@ -1301,7 +1300,7 @@ handle (request& rq, response& rs)
                 {
                   b = make_shared<build> (move (bid.package.tenant),
                                           move (bid.package.name),
-                                          move (bp.version),
+                                          p->version,
                                           move (bid.target),
                                           move (bid.target_config_name),
                                           move (bid.package_config_name),
-- 
cgit v1.1