Contributing factor in T192591 - We had the jobs shut down for more hours than requeue-max-age, so when we threw the first RetryableException, the message was already deemed to be too old to retry. We used the original enqueued date as a way to not clutter the message too much, since we're mixing body fields with what used to be headers. We should just reintroduce a field for retry count.
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Use a new field to track retry count instead of age | wikimedia/fundraising/SmashPig | master | +6 -6 |
Event Timeline
Comment Actions
Change 428032 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/SmashPig@master] Use a new field to track retry count instead of age
Comment Actions
Change 428032 abandoned by Ejegg:
[wikimedia/fundraising/SmashPig@master] Use a new field to track retry count instead of age
Reason: