malbrec2/cctools — pull request #1 by dpandiar (2013-04-17 01:38:02+00:00)
Title: Currently, we don't catch failures when processing a received message in recv_worker_msg(). This fix changes recv_worker_msg() to return failure if the processing fails.

Mike-

In my runs, I notice some spurious workers connect. They don't have a project name and the master rightfully determines that their project does not match with its. But then it does not bail on the worker. It actually continues to send input files to such workers where it fails because the string interpolation for file names fails.

The problem is we don't currently bail if the processing of a received message fails. I don't know if this is a long-term fix since we might want to handle failures directly in the individual process message functions.

Currently, we don't catch failures when processing a received message in
recv_worker_msg(). This fix changes recv_worker_msg() to return failure if the
processing fails.