Fix flakiness dashboard stability and performance issues.
authordburkart@apple.com <dburkart@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Aug 2015 21:35:54 +0000 (21:35 +0000)
committerdburkart@apple.com <dburkart@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Aug 2015 21:35:54 +0000 (21:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=147835

Reviewed by Ryosuke Niwa.

* init-database.sql:
* public/.htaccess:
* public/include/json-shared.php:
* public/include/test-results.php:

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

Websites/test-results/ChangeLog
Websites/test-results/init-database.sql
Websites/test-results/public/.htaccess
Websites/test-results/public/include/json-shared.php
Websites/test-results/public/include/test-results.php

index fb215c5..9b31926 100644 (file)
@@ -1,3 +1,15 @@
+2015-08-10  Dana Burkart  <dburkart@apple.com>
+
+        Fix flakiness dashboard stability and performance issues.
+        https://bugs.webkit.org/show_bug.cgi?id=147835
+
+        Reviewed by Ryosuke Niwa.
+
+        * init-database.sql:
+        * public/.htaccess:
+        * public/include/json-shared.php:
+        * public/include/test-results.php:
+
 2014-01-23  Ryosuke Niwa  <rniwa@webkit.org>
 
         Upstream changes to json-shared.php from the perf dashboard
         (TestResultsView._createResultCell): Show the test time and the expected result.
         (TestResultsView._createTestResultRow): Compute the slowest run and also round time to tenth of second for time
         less than 10s or second if it's more than 10s so that the test time will always be shown in two digits.
-        Also show the bug number and the latest expected result on the left columns after linkifying the bug numbers. 
+        Also show the bug number and the latest expected result on the left columns after linkifying the bug numbers.
         (TestResultsView._matchesFailureType): Added. Determines whether results is of a particular failure type.
         (TestResultsView._populateBuilderPane):
         (TestResultsView.fetchFailingTestsForBuilder): Store the failure type such as flaky, wrongtestexpectations.
         loadTestsFromLocationHash.
         (TestResultsView.loadTestsFromLocationHash): Take care of both 'tests' and 'builder' components.
         (fetchManifest): Setup the UI to select a builder.
-
index 0ff6e12..f7821f7 100644 (file)
@@ -51,7 +51,7 @@ CREATE TABLE tests (
     reftest_type varchar(64));
 
 CREATE TABLE results (
-    id serial PRIMARY KEY,
+    id bigserial PRIMARY KEY,
     test integer REFERENCES tests ON DELETE CASCADE,
     build integer REFERENCES builds ON DELETE CASCADE,
     expected varchar(64) NOT NULL,
@@ -64,4 +64,4 @@ CREATE INDEX results_test ON results(test);
 CREATE INDEX results_build ON results(build);
 CREATE INDEX results_is_flaky ON results(is_flaky);
 
-SET work_mem='50MB';
+SET work_mem='1024MB';
index 817aa47..8a34643 100644 (file)
@@ -5,4 +5,5 @@ php_value upload_max_filesize 100000000
 php_value post_max_size 110000000
 php_value memory_limit 120000000
 php_value max_input_time 60
+php_value max_execution_time 240
 </IfModule>
index 5100603..f53b19b 100644 (file)
@@ -1,4 +1,5 @@
 <?php
+ini_set('memory_limit', '1024M');
 
 require_once('db.php');
 
@@ -67,7 +68,7 @@ function recursively_add_test_results($db, $build_id, $tests, $full_name) {
     require_format('test_time', $tests['time'], '/^\d*$/');
     $modifiers = array_get($tests, 'modifiers');
     if ($modifiers)
-        require_format('test_modifiers', $modifiers, '/^[A-Za-z0-9 \.\/]+$/');
+        require_format('test_modifiers', $modifiers, '/^[A-Za-z0-9 \.\/\+]+$/');
     else
         $modifiers = NULL;
     $category = 'LayoutTest'; // FIXME: Support other test categories.