← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] lp:~cjwatson/launchpad/remove-tickcount into lp:launchpad

 

Colin Watson has proposed merging lp:~cjwatson/launchpad/remove-tickcount into lp:launchpad.

Commit message:
Remove tickcount and track traversal/publication durations instead.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/remove-tickcount/+merge/373347

tickcount can't work on Python 3, so in the long term we don't have much choice here.  We do want to be able to spot contention, but tracking thread time should be enough here.  (Unfortunately, as far as I can tell this is only possible with Python >= 3.3.)

This does change the access log format slightly (logging durations rather than ticks), but as far as I can tell nothing cares.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~cjwatson/launchpad/remove-tickcount into lp:launchpad.


Follow ups