HomePhorge

Recover more cleanly from worker tasks with unconstructable classes

Description

Recover more cleanly from worker tasks with unconstructable classes

Summary:
This is unusual, but if getWorkerInstance() throws we end up with an undefined $worker when recovering from the exception.

Instead, handle this case slightly more gracefully.

The easiest way to hit this is to schedule a task for a worker that doesn't exist (or remove an existing worker, which is what I did to hit it).

Test Plan: Saw a more graceful error recovery; ran some normal successful tasks out of the queue.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D11413

Details

Provenance
epriestleyAuthored on Jan 15 2015, 3:57 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP9f29af108bbb: Fix visibility of some `LiskDAO` methods
Branches
Unknown
Tags
Unknown

Event Timeline