The proposal to merge lp:~mbp/launchpad/show-timeline into lp:launchpad has 
been updated.

Description changed to:

With this change, you can click the "%d queries/external actions" show to 
developers to actually see what the queries were.  An HTML form of the request 
timeline is revealed at the bottom of the page.  I think I would find this 
interesting or useful on pages that are slow but not failing, or that are 
intermittently oops.

I realize you can force it to oops, wait for the oops to sync, then look at it 
elsewhere but having the data right there seems easier and perhaps more likely 
to actually be inspected.  Also, because performance depends on cache hits, 
retrying the request to get an oops may give different behaviour.

This just renders data already collected in the timeline so it should not 
affect performance very much.

It's not automatically tested yet.  Pointers to where the tests ought to be 
added would be welcome.

I thought about highlighting the slow actions but I haven't done that either.

Clicking the link reveals the list but doesn't scroll to it; I have mixed 
feelings about whether it should, and in any case making the anchor the target 
and the javascript show it from onclick didn't seem to work on the first 
pageload.  Advice welcome.

screenshots:
  http://people.canonical.com/~mbp/launchpad-screenshots/20111024_006.png
  http://people.canonical.com/~mbp/launchpad-screenshots/20111024_007.png


For more details, see:
https://code.launchpad.net/~mbp/launchpad/show-timeline/+merge/80166
-- 
https://code.launchpad.net/~mbp/launchpad/show-timeline/+merge/80166
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~mbp/launchpad/show-timeline into lp:launchpad.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to