On Wed, Mar 23, 2011 at 04:41:02PM +0100, Michael Hanselmann wrote:
> Am 23. März 2011 15:27 schrieb Iustin Pop <[email protected]>:
> > On Wed, Mar 23, 2011 at 03:00:55PM +0100, Michael Hanselmann wrote:
> >> the actual code in SubmitManyJobs uses the
> >> string representation (“str(err)”) for failed submissions. If you
> >> think that's still not enough SubmitManyJobs should be extended. What
> >> do you think?
> >
> > I think that beside the reason for failure (i.e. str(err)), we need to
> > know what the job that failed contained, hence my suggestion to list the
> > summary of the jobs.
> >
> > That can be done from the caller of SubmitManyJobs, without changing the
> > API.
> 
> Both ways work for me. I think I'll just change SubmitManyJobs to
> include the summary in the error message—that way it'll be available
> for direct callers of SubmitManyJobs, too.

Sure. LGTM.

iustin

Reply via email to