Merge pull request #1831 from s7tya/allow-no-current-builds

To allow the `/perf/status_page` to run even if the current collection is empty

Author: Kobzol

database/src/pool/sqlite.rs

@@ -1096,7 +1096,7 @@ impl Connection for SqliteConnection {
                 "
                 select
                     step,
-                    end_time is not null,
+                    end is not null,
                     coalesce(end, strftime('%s', 'now')) - start,
                     (select end - start
                         from collector_progress as cp
@@ -1114,8 +1114,8 @@ impl Connection for SqliteConnection {
                 Ok(crate::Step {
                     name: row.get(0)?,
                     is_done: row.get(1)?,
-                    duration: Duration::from_secs(row.get::<_, i64>(2)? as u64),
-                    expected: Duration::from_secs(row.get::<_, i64>(3)? as u64),
+                    duration: Duration::from_secs(row.get::<_, i64>(2).unwrap_or_default() as u64),
+                    expected: Duration::from_secs(row.get::<_, i64>(3).unwrap_or_default() as u64),
                 })
             })
             .unwrap()

site/src/load.rs

@@ -454,12 +454,11 @@ fn sort_queue(
         let level_len = partition_in_place(unordered_queue[finished..].iter_mut(), |(_, mr)| {
             mr.parent_sha().map_or(true, |parent| done.contains(parent))
         });
-        assert!(
-            level_len != 0,
-            "at least one commit is ready done={:#?}, {:?}",
-            done,
-            &unordered_queue[finished..]
-        );
+
+        if level_len == 0 {
+            return vec![];
+        }
+
         let level = &mut unordered_queue[finished..][..level_len];
         level.sort_unstable_by_key(|(c, mr)| {
             (