thieman/dagobah — pull request #105 by rclough (2014-07-10 23:06:25+00:00)
Title: JobTask subclass

The goal of this PR is to come down to an acceptable implementation of the JobTask subclass

I pasted over mostof the methods from the Task class and added exceptions to them, so that, at least for the implementation of Jobs-in-Jobs, dagobah will scream if we try to treat a JobTask like a regular task.

The idea is simple, Job task is simply a placeholder with the name of the Job it's referencing, os that when we execute the job, the tasks will be expanded under the hood for actual runtime. Thus this class really only needs the referencing to it's job.

Things to consider: do we want this class to have run logs? Soft/Hard timeouts? I lean towards thingking the logs and timeouts should be kept with the actual job underlying.

Expansion of the JobTask into subtasks will require renaming tasks, as, probably prepending them with the JobTask name, because theoretically you could reference the same job twice in one script.

Is there any other effects of adding this implementation you think should be added into the PR?