Unreviewed build fix after r174477.
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Oct 2014 00:19:14 +0000 (00:19 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Oct 2014 00:19:14 +0000 (00:19 +0000)
* init-database.sql: Removed build_commits_index since it's redundant with build_commit's primary key.
Also fixed a syntax error that we were missing "," after line that declared build_commit column.

* public/api/runs.php: Fixed the query so that test_runs without commits data will be retrieved.
This is necessary for baseline and target values manually added via admin pages.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@174486 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/init-database.sql
Websites/perf.webkit.org/public/api/runs.php

index c62e1851c05e79b4209d0cff81d8c740cddeabb6..0d28e78b65d9c6d90c0927790dfd10f963d6fffd 100644 (file)
@@ -1,3 +1,13 @@
+2014-10-08  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Unreviewed build fix after r174477.
+
+        * init-database.sql: Removed build_commits_index since it's redundant with build_commit's primary key.
+        Also fixed a syntax error that we were missing "," after line that declared build_commit column.
+
+        * public/api/runs.php: Fixed the query so that test_runs without commits data will be retrieved.
+        This is necessary for baseline and target values manually added via admin pages.
+
 2014-10-08  Ryosuke Niwa  <rniwa@webkit.org>
 
         Add v2 UI for the perf dashboard
index 148787e63656c84ee8714d0f14b6cd9dc9ffcc66..fb1f2443803ff21f1303e14eb7f96388411dc3ed 100644 (file)
@@ -68,9 +68,8 @@ CREATE INDEX commit_author_email_index ON commits(commit_author_email);
 
 CREATE TABLE build_commits (
     commit_build integer NOT NULL REFERENCES builds ON DELETE CASCADE,
-    build_commit integer NOT NULL REFERENCES commits ON DELETE CASCADE
+    build_commit integer NOT NULL REFERENCES commits ON DELETE CASCADE,
     PRIMARY KEY (commit_build, build_commit));
-CREATE INDEX build_commits_index ON build_commits(commit_build, build_commit);
 
 CREATE TABLE aggregators (
     aggregator_id serial PRIMARY KEY,
index d06a5fc4b068f5cbc4a42080af7a3d75a1d806d0..a059848f5914630e55301ff4d9b33c862c9844cb 100644 (file)
@@ -31,8 +31,8 @@ if ($repository_table = $db->fetch_table('repositories')) {
 function fetch_runs_for_config($db, $config) {
     $raw_runs = $db->query_and_fetch_all('
     SELECT test_runs.*, builds.*, array_agg((commit_repository, commit_revision, commit_time)) AS revisions
-        FROM builds LEFT OUTER JOIN build_commits ON commit_build = build_id, test_runs, commits
-        WHERE run_build = build_id AND run_config = $1 AND build_commit = commit_id
+        FROM builds LEFT OUTER JOIN build_commits ON commit_build = build_id LEFT OUTER JOIN commits ON build_commit = commit_id, test_runs
+        WHERE run_build = build_id AND run_config = $1
         GROUP BY build_id, run_id', array($config['config_id']));
 
     $formatted_runs = array();