Don't fetch more than two builds to check duplicity of builds in ReportProcessor
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jan 2018 23:01:46 +0000 (23:01 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jan 2018 23:01:46 +0000 (23:01 +0000)
https://bugs.webkit.org/show_bug.cgi?id=181786

Reviewed by Wenson Hsieh.

Limit the number of builds to check to avoid using a lot of memory with a long delay.

* public/include/report-processor.php:
(ReportProcessor::resolve_build_id):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/include/report-processor.php

index ed2b1fb..655ded8 100644 (file)
@@ -1,3 +1,15 @@
+2018-01-18  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Don't fetch more than two builds to check duplicity of builds in ReportProcessor
+        https://bugs.webkit.org/show_bug.cgi?id=181786
+
+        Reviewed by Wenson Hsieh.
+
+        Limit the number of builds to check to avoid using a lot of memory with a long delay.
+
+        * public/include/report-processor.php:
+        (ReportProcessor::resolve_build_id): 
+
 2018-01-17  Ryosuke Niwa  <rniwa@webkit.org>
 
         Rename config.json to config.json.sample
index 2fc2f5c..ca1a031 100644 (file)
@@ -130,7 +130,7 @@ class ReportProcessor {
     private function resolve_build_id(&$build_data, $revisions, $build_request_id) {
         // FIXME: This code has a race condition. See <rdar://problem/15876303>.
         $results = $this->db->query_and_fetch_all("SELECT build_id, build_slave FROM builds
-            WHERE build_builder = $1 AND build_number = $2 AND build_time <= $3 AND build_time + interval '1 day' > $3",
+            WHERE build_builder = $1 AND build_number = $2 AND build_time <= $3 AND build_time + interval '1 day' > $3 LIMIT 2",
             array($build_data['builder'], $build_data['number'], $build_data['time']));
         if ($results) {
             $first_result = $results[0];