1033328 - Toggling "show only unclassified failures" is slower than TBPL

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Ed Morley [:emorley]]

1) Go to https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound
2) Wait for the page to fully load
3) Press the 'u' key (to toggle showing only unclassified failures)
4) Observe time taken for the UI to update
5) Repeat steps 2-4 with https://tbpl.mozilla.org/?tree=Mozilla-Inbound

Expected:
Same time or quicker than TBPL for the UI to update

Actual:
TBPL updates virtually instantly (fast enough it's hard to time). Treeherder takes~2 seconds.

Comment 1

[Cameron Dawson [:camd]]

This is fixed in https://github.com/mozilla/treeherder-ui/commit/dbc7d5c22481df76057289d53649dc61aab6afd0

As the commit message states, this may not be quite as fast as TBPL, but should bring the speed into an acceptable range.  The speed will totally depend on the number of jobs in the visible resultsets.  For the default of 10 on Mozilla Central when there are a huge number of jobs, we are now less than 1 second.  Fewer jobs will be much faster.

Comment 2

[Cameron Dawson [:camd]]

Attachment: pull request

Comment 3

[Ed Morley [:emorley]]

Much better, thank you :-)

Comment 4

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/6678e359c84e73a8778d2744444cf8e60e55f77c
fix bug 1033328: Class based filtering with jquery selectors now working

Also fixes bug 1033330 and bug 1043783 as they were related to the work for this perf improvement.
The speed improvements here may not be quite as fast as TBPL, but they should be at an acceptable level.
This changes our filtering mechanism to use JQuery selectors to set jobs, groups and platforms vislble or not.
This also contains some refactoring to make some functions more reusable and prevent circular references in our classes.