Title: [227166] trunk/Websites/perf.webkit.org
- Revision
- 227166
- Author
- [email protected]
- Date
- 2018-01-18 15:01:46 -0800 (Thu, 18 Jan 2018)
Log Message
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):
Modified Paths
Diff
Modified: trunk/Websites/perf.webkit.org/ChangeLog (227165 => 227166)
--- trunk/Websites/perf.webkit.org/ChangeLog 2018-01-18 22:43:01 UTC (rev 227165)
+++ trunk/Websites/perf.webkit.org/ChangeLog 2018-01-18 23:01:46 UTC (rev 227166)
@@ -1,3 +1,15 @@
+2018-01-18 Ryosuke Niwa <[email protected]>
+
+ 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 <[email protected]>
Rename config.json to config.json.sample
Modified: trunk/Websites/perf.webkit.org/public/include/report-processor.php (227165 => 227166)
--- trunk/Websites/perf.webkit.org/public/include/report-processor.php 2018-01-18 22:43:01 UTC (rev 227165)
+++ trunk/Websites/perf.webkit.org/public/include/report-processor.php 2018-01-18 23:01:46 UTC (rev 227166)
@@ -130,7 +130,7 @@
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];
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
