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 c62e185..0d28e78 100644 (file)
@@ -1,5 +1,15 @@
 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
         https://bugs.webkit.org/show_bug.cgi?id=137537
 
index 148787e..fb1f244 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 d06a5fc..a059848 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();