Bug 123797

Summary: New flakiness dashboard should generate JSON in the background process
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: WebKit WebsiteAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, ap, kling, lforschler, mrowe, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch ap: review+

Description Ryosuke Niwa 2013-11-05 04:03:01 PST
Don't generate update flakiness state or generate JSONs in the foreground process as that could cause a resource starvation.
Comment 1 Ryosuke Niwa 2013-11-05 04:13:39 PST
Created attachment 216021 [details]
Patch
Comment 2 Alexey Proskuryakov 2013-11-05 23:38:37 PST
Comment on attachment 216021 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=216021&action=review

> Websites/test-results/public/admin/process-builds.php:8
> +function processed_builds($db) {

I think that this function could use a better name. Is it actually called after builds are processed, as the name implies? All or some of them?
Comment 3 Ryosuke Niwa 2013-11-05 23:50:24 PST
(In reply to comment #2)
> (From update of attachment 216021 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=216021&action=review
> 
> > Websites/test-results/public/admin/process-builds.php:8
> > +function processed_builds($db) {
> 
> I think that this function could use a better name. Is it actually called after builds are processed, as the name implies? All or some of them?

Oops, it should read process_latest_five_builds instead.  I apparently forgot to rename :(
Comment 4 Ryosuke Niwa 2013-11-06 19:47:34 PST
Committed r158814: <http://trac.webkit.org/changeset/158814>