Review: Approve

Looks good.

[0]

63      +        sleep(1)

I see you haven't changed that but I'm wondering if we shouldn't use a more 
fine-grained value than one second.  I don't have any data to know if this 
would gain us much but if we're going to use 'wait_for_queue' over and over 
again, that might be worth checking.
-- 
https://code.launchpad.net/~abentley/launchpad/enable-tests/+merge/116678
Your team Launchpad code reviewers is subscribed to branch 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