Page MenuHomePhabricator

Remove retry/failure mechanisms from MetaMTA
ClosedPublic

Authored by epriestley on Jan 30 2014, 11:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 8:39 AM
Unknown Object (File)
Sat, Apr 6, 1:27 PM
Unknown Object (File)
Sat, Apr 6, 7:46 AM
Unknown Object (File)
Thu, Apr 4, 11:19 AM
Unknown Object (File)
Mar 20 2024, 12:49 AM
Unknown Object (File)
Mar 20 2024, 12:49 AM
Unknown Object (File)
Mar 20 2024, 12:03 AM
Unknown Object (File)
Mar 19 2024, 11:40 PM
Subscribers
Tokens
"Mountain of Wealth" token, awarded by dctrwatson.

Details

Summary

Fixes T4202. We have old code in MetaMTA which implements gradual backoff and maximum retries.

However, we have more general code in the task queue which does this, too. We can just use the more general stuff in the task queue; it obsoletes the specific stuff in MetaMTA, which is more complex and ran into some kind of issue in T4202.

Remove retryCount, nextRetry (obsoleted by task queue retry mechanisms) and "simulated failures" (no longer in use).

Generally, modern infrastructure has replaced these mechanisms with more general ones.

Test Plan
  • Sent mail.
  • Observed unsendable mail failing in reasonable ways in the queue.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Nice. I especially like the DB cleanup...!