Page MenuHomePhabricator

Treat PHP7 "Throwable" exceptions like other unhandled "Exception" cases in the worker queue
ClosedPublic

Authored by epriestley on May 19 2020, 5:39 PM.
Tags
None
Referenced Files
F12838939: D21271.id50654.diff
Thu, Mar 28, 6:53 PM
Unknown Object (File)
Sun, Mar 24, 8:29 PM
Unknown Object (File)
Sat, Mar 23, 1:02 PM
Unknown Object (File)
Tue, Mar 19, 1:43 AM
Unknown Object (File)
Tue, Mar 5, 4:53 AM
Unknown Object (File)
Feb 26 2024, 12:28 AM
Unknown Object (File)
Feb 25 2024, 12:49 AM
Unknown Object (File)
Feb 16 2024, 8:26 PM
Subscribers
None

Details

Summary

See PHI1745. See T12855. Under PHP7, errors raised as Throwable miss this "generic exception" logic and don't increment their failure count. Instead, treat any "Throwable" we don't recognize like any "Exception" we don't recognize.

Test Plan
  • Under PHP7, caused a worker task to raise a Throwable (e.g., call to undefined method, see D21270).
  • Ran bin/worker execute --id ....
  • Before: worker failed, but did not increment failure count.
  • After: worker fails and increments failure count as it would for other types of unknown error.

Diff Detail

Repository
rP Phabricator
Branch
throwable1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24470
Build 33719: Run Core Tests
Build 33718: arc lint + arc unit