Code review comment for lp:~cjwatson/python-oops-amqp/publisher-handle-channel-errors

Revision history for this message
Colin Watson (cjwatson) wrote :

I do have some concerns about what would happen if the broker was up but slow and there was an OOPS storm: we'd incur an extra second in each case. I added this mainly because it was difficult to reproduce the problem in a test case otherwise. On the other hand, an extra second isn't terrible in that sort of situation, and might even allow the broker to recover more quickly by slowing down the input slightly.

« Back to merge proposal