If we're supporting multiple output types (xml + csv), the name of this method is a bit misleading. Considering that this method is doing, I would expect a name more like `get_outputs_for_job`, or something. In fact, why not just move this to `Output`s manager?