Bug 151529 - Internal perf dashboard loads ~30MB of data
Summary: Internal perf dashboard loads ~30MB of data
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Perf Dashboard (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on: 151582
Blocks:
  Show dependency tree / graph
 
Reported: 2015-11-20 16:58 PST by Ryosuke Niwa
Modified: 2017-03-10 19:57 PST (History)
5 users (show)

See Also:

Attachments
Fixes the bug (5.94 KB, patch)
2015-11-20 17:12 PST, Ryosuke Niwa 		timothy: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2015-11-20 16:58:32 PST 
Here's yet another JSON API optimization.

Don't include data more than 366 days old.
Comment 1 Ryosuke Niwa 2015-11-20 17:12:31 PST 
Created attachment 266022 [details]
Fixes the bug
Comment 2 Ryosuke Niwa 2015-11-20 19:12:16 PST 
Committed r192716: <http://trac.webkit.org/changeset/192716>
Comment 3 WebKit Commit Bot 2015-11-23 22:51:46 PST 
Re-opened since this is blocked by bug 151582
Comment 4 Ryosuke Niwa 2015-11-24 01:00:52 PST 
Comment on attachment 266022 [details]
Fixes the bug

Alright, I need to change my approach. Change the format of JSON API can reduce the file size from 6.7MB to 2.2MB.

Unfortunately, that requires a lot of front-end work so let me finish the rewrite of the interactive chart component first.
Comment 5 Ryosuke Niwa 2015-11-24 02:42:31 PST 
Hm... I can't quite do that either unless we want to create hundreds of tiny HTTP to fetch commit data :(
Comment 6 Ryosuke Niwa 2017-03-10 19:57:55 PST 
This has been fixed in v3 API.