Issue 692: Diff viewer pagination is absurdly silly
http://code.google.com/p/reviewboard/issues/detail?id=692

New issue report by trowbrds:
When we paginate things onto the diff viewer, we do it at the wrong time in
the process. What we do today is:
1. For each file in the diff
     a. Fetch the file from the SCM
     b. Patch the file
2. Diff
3. Paginate
4. Render

Rendering can be an expensive step, but not that expensive. Really what we
should be doing is:
1. Paginate
2. For each file on the page
     a. Fetch the file from the SCM
     b. Patch the file
3. Diff
4. Render


Issue attributes:
        Status: Confirmed
        Owner: ----
        Labels: Type-Defect Priority-High Component-DiffViewer Performance

-- 
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to