[
https://issues.apache.org/jira/browse/GIRAPH-576?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jess Garms updated GIRAPH-576:
------------------------------
Attachment: GIRAPH-576.patch
Updated patch allowing callers to set the reason when failing a job.
> BspServiceMaster.failureCleanup() shouldn't pass null in observers'
> applicationFailed() method
> ----------------------------------------------------------------------------------------------
>
> Key: GIRAPH-576
> URL: https://issues.apache.org/jira/browse/GIRAPH-576
> Project: Giraph
> Issue Type: Bug
> Components: bsp
> Affects Versions: 0.2.0
> Environment: Linux
> Reporter: Jess Garms
> Priority: Minor
> Labels: easy, newbie, patch
> Fix For: 0.2.0
>
> Attachments: GIRAPH-576.patch
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> failureCleanup() in BspServiceMaster gets called with a null exception from
> failJob(). That in turn passes a null exception to the set of
> MasterObservers, in their applicationFailed() method. They probably aren't
> expecting that. Instead we should pass an appropriate exception around
> depending on the cause of the failure.
> I'll attach a patch.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
