From d1ca6392a4b80f19b467dea14908c6fc9d0f7ff8 Mon Sep 17 00:00:00 2001
From: Oliver Giles <ohw.giles@gmail.com>
Date: Fri, 1 Jun 2018 10:33:25 +0300
Subject: [PATCH] resolves #43: limit the results displayed on home page

Limit "runs per job in last 24h" and "average runtime per job"
graphs to the top 5 in each case
---
 src/laminar.cpp          | 4 ++--
 src/resources/index.html | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/laminar.cpp b/src/laminar.cpp
index aec1950..713566d 100644
--- a/src/laminar.cpp
+++ b/src/laminar.cpp
@@ -333,14 +333,14 @@ void Laminar::sendStatus(LaminarClient* client) {
         }
         j.EndArray();
         j.startObject("buildsPerJob");
-        db->stmt("SELECT name, COUNT(*) FROM builds WHERE completedAt > ? GROUP BY name")
+        db->stmt("SELECT name, COUNT(*) c FROM builds WHERE completedAt > ? GROUP BY name ORDER BY c DESC LIMIT 5")
                 .bind(time(nullptr) - 86400)
                 .fetch<str, int>([&](str job, int count){
             j.set(job.c_str(), count);
         });
         j.EndObject();
         j.startObject("timePerJob");
-        db->stmt("SELECT name, AVG(completedAt-startedAt) FROM builds WHERE completedAt > ? GROUP BY name")
+        db->stmt("SELECT name, AVG(completedAt-startedAt) av FROM builds WHERE completedAt > ? GROUP BY name ORDER BY av DESC LIMIT 5")
                 .bind(time(nullptr) - 7 * 86400)
                 .fetch<str, uint>([&](str job, uint time){
             j.set(job.c_str(), time);
diff --git a/src/resources/index.html b/src/resources/index.html
index 0ed94f3..95905be 100644
--- a/src/resources/index.html
+++ b/src/resources/index.html
@@ -91,7 +91,7 @@
    <div class="col-sm-7 col-md-8 col-lg-9"><div class="row">
     <div class="col-md-6">
      <div class="panel panel-default">
-      <div class="panel-heading">Total builds per day this week</div>
+      <div class="panel-heading">Total runs per day this week</div>
       <div class="panel-body">
        <canvas id="chartBpd"></canvas>
       </div>
@@ -99,7 +99,7 @@
     </div>
     <div class="col-md-6">
      <div class="panel panel-default">
-      <div class="panel-heading">Builds per job in the last 24 hours</div>
+      <div class="panel-heading">Most runs per job in the last 24 hours</div>
       <div class="panel-body" id="chartStatus">
        <canvas id="chartBpj"></canvas>
       </div>
@@ -107,7 +107,7 @@
     </div>
     <div class="col-md-6">
      <div class="panel panel-default">
-      <div class="panel-heading">Average build time per job this week</div>
+      <div class="panel-heading">Longest average run time per job this week</div>
       <div class="panel-body">
        <canvas id="chartTpj"></canvas>
       </div>