tree ff71611ac49cd4b9234aeaad7cccd73092e5eef5
parent efe8656512373389771aff88c2141940f925ad58
author Santhosh Kumar Shanmugham <santhoshkumar.s@gmail.com> 1531952607 -0700
committer Santhosh Kumar <sshanmugham@twitter.com> 1531952607 -0700

Unhandled exception should not strand runner in STARTING state.

If the ThermoTaskRunner encounters an Exception when trying to
fork the process, it bubbles this up to the Executor which does
not handle execptions other than TaskError. This leads to the
executor leaving the task in STARTING state and we end up with
tasks that get stranded in this state.

Fix it so that any unknown expection that is thrown when starting
a runner leads to task failure and get marked as FAILED.

Testing Done:
./gradlew test
./pants test src/test/python/apache::

Reviewed at https://reviews.apache.org/r/67967/