launchpad-reviewers team mailing list archive

[Raphaël Badin <raphael.badin@xxxxxxxxxxxxx>]

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.