[ 
https://issues.apache.org/jira/browse/MAHOUT-946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13203694#comment-13203694
 ] 

tom pierce commented on MAHOUT-946:
-----------------------------------

I agree that the cleanup stuff seems a bit awkward as is.

I'd like to see the checks get in, and I think halting execution on job failure 
is something folks can agree is an improvement.  It might be harder to settle 
on consensus for job cleanup strategies.  

Would it make sense at this point to integrate the earlier patch, without the 
cleanups, and split off utility methods for cleanup to another ticket?  If so 
I'll volunteer to open a new issue and submit an initial patch to address the 
cleanup issue.

                
> Map-reduce job status often left unchecked
> ------------------------------------------
>
>                 Key: MAHOUT-946
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-946
>             Project: Mahout
>          Issue Type: Bug
>    Affects Versions: 0.6
>            Reporter: tom pierce
>         Attachments: MAHOUT-946.patch, MAHOUT-946.patch
>
>
> I've run into a few places in Mahout where mapreduce jobs can fail and their 
> status won't be checked, so processing will continue on.  This sometimes 
> obscures the root problem.  I've tracked down a bunch of places where this 
> problem exists, and tried to fix the code to do something reasonable (return 
> -1 status code, throw exception, etc.) when jobs fail.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to