Bug 157827

Summary: Botwatcher's dashboard should cache JSON results using IndexedDB if available
Product: WebKit Reporter: Dana Burkart <dburkart>
Component: Tools / TestsAssignee: Dana Burkart <dburkart>
Status: ASSIGNED    
Severity: Normal CC: aakash_jain, ap, beidson, lforschler, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch beidson: review-

Dana Burkart
Reported 2016-05-17 21:54:35 PDT
The botwatcher's dashboard makes a huge amount of XHR requests to buildbot in order to load the latest passing / failing build for a given queue. Instead, if the user is using a browser which supports IndexedDB, the dashboard should use that to cache results. This would help with performance, especially in instances where the dashboard has to talk to a buildbot instance with tons of build queues.
Attachments
Patch (6.53 KB, patch)
2016-06-22 11:40 PDT, Dana Burkart 		beidson: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dana Burkart
Comment 1 2016-06-22 11:40:53 PDT
Created attachment 281853 [details] Patch This patch adds a new option to the JSON.load function to instruct it to cache things in IndexedDB.
Dana Burkart
Comment 2 2016-06-22 11:41:22 PDT
rdar://problem/26096900
Brady Eidson
Comment 3 2017-08-19 16:02:23 PDT
Comment on attachment 281853 [details] Patch r-, as this has been pending review for over a year now. It is near-impossible that this patch still applies to trunk and unlikely to still be relevant in its current form.
Alexey Proskuryakov
Comment 4 2017-08-19 17:12:33 PDT
We can hopefully optimize for buildbot 0.9 without the need for local caching.
Note You need to log in before you can comment on or make changes to this bug.