Diffs for big patches take too long to load

NEW
Unassigned

Status

MozReview
Review Board: DiffViewer
a year ago
7 months ago

People

(Reporter: Gijs, Unassigned)

Tracking

(Blocks: 1 bug)

Details

+++ This bug was initially created as a clone of Bug #1288112 +++

STR:

load https://reviewboard.mozilla.org/r/28271/diff/1#index_header

This is not even a terribly large patch, it's even worse with something like:

https://reviewboard.mozilla.org/r/40859/diff/4/#index_header


ER:

the patch loads promptly, with a minimum of obvious stalling/spinner-ing/loading


AR:
every individual file at the top of the page has a spinner that takes multiple seconds per file to disappear. All the files themselves are also loaded dynamically, each taking ages to load. Even now that the cache on the server is hot and so a ctrl-shift-reload is faster than the initial load when I filed this bug, it takes a full 20 seconds before the page has finished. It was more like several minutes the first time. That's not acceptable performance.
Blocks: 1288133
Component: Review Board: User Interface → Review Board: DiffViewer
You need to log in before you can comment on or make changes to this bug.