Code review comment for lp:~thomir-deactivatedaccount/core-result-checker/trunk-check-swift-better

Revision history for this message
Celso Providelo (cprov) wrote :

Right, let's see the SwiftManager in action.

I just think that 10 minutes spinning message waiting for swift seems excessive (I can see the logging storm already), but we shall see ...

For situations like this I'd prefer moving jobs quickly (< 30 s) to the deadletter queue and have proper monitoring and replay scripts.

review: Approve

« Back to merge proposal