[ 
https://issues.apache.org/jira/browse/MAPREDUCE-6927?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Badger updated MAPREDUCE-6927:
-----------------------------------
    Attachment: MAPREDUCE-6927.004.patch

Thanks for the additional review. Thought I got rid of all of the checkstyle 
issues, but I must've added in new ones as I updated the patch. I'm pretty sure 
this one got rid of them all. I had to move {{moveTmpToDone}} to protected so 
that I could override it in the test. Not sure if there's another way around 
that. 

> MR job should only set tracking url if history was successfully written
> -----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6927
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6927
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Eric Badger
>            Assignee: Eric Badger
>         Attachments: MAPREDUCE-6927.001.patch, MAPREDUCE-6927.002.patch, 
> MAPREDUCE-6927.003.patch, MAPREDUCE-6927.004.patch
>
>
> Currently the RMCommunicator will set the tracking url during unregistration 
> once a job has finished, regardless of whether it actually wrote history or 
> not. If the write to history failed for whatever reason, we should leave the 
> tracking url as null so that we get redirected to the AHS instead of getting 
> a job not found on the JHS. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to