Don't need to download jobtype and jobgroup information on load

RESOLVED FIXED

Status

Tree Management
Treeherder
RESOLVED FIXED
7 months ago
6 months ago

People

(Reporter: wlach, Assigned: wlach)

Tracking

Details

Attachments

(1 attachment)

We currently download the jobtype and jobgroup information every time we load treeherder to establish a "sorting order" for this information. 

This is rather slow in the case of job type information (3 seconds from my local wifi network) and in point of fact we don't need to do this -- we can just sort the job and group information dynamically using the same algorithm and have the same result.

With this enabled, treeherder is a good chunk faster to load initially, most especially on things like try pushes where downloading the jobtype information was actually the bottleneck!

Comment 1

7 months ago
Created attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master
Comment on attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master

Noticed this while working on some other stuff. This isn't urgent, but does provide a nice speed-up when loading treeherder.
Attachment #8799247 - Flags: review?(cdawson)
Comment on attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master

Hey Will--  this is great!  Just a couple of minor cleanup things here and there.  :)
Attachment #8799247 - Flags: review?(cdawson) → review-
Comment on attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master

Made the changes you requested, could you have a quick look?
Attachment #8799247 - Flags: review- → review?(wlachance)
Comment on attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master

Gah, I meant to r? camd, not myself. :)
Attachment #8799247 - Flags: review?(wlachance) → review?(cdawson)
Comment on attachment 8799247 [details] [review]
[treeherder] wlach:1308782 > mozilla:master

Awesome!  :)
Attachment #8799247 - Flags: review?(cdawson) → review+

Comment 7

6 months ago
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/700ef6bec2b681fdc564de25ada1f3a954cba521
Bug 1308782 - Don't download job and group data on load (#1912)

Instead, just calculate on the fly with the job information we have.
This makes the display of the first job data about 3 seconds faster in
my preliminary testing.
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.