[
https://issues.apache.org/jira/browse/MAPREDUCE-6927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116712#comment-16116712
]
Jason Lowe commented on MAPREDUCE-6927:
---------------------------------------
Thanks for updating the patch! It's really close.
The moveTmpToDone changes are frivolous since nobody is checking the return
value. I think we can leave those changes out.
Since this is touching the "copy failed" log message, it would be nice if we
updated that message to be more specific on the source and dest involved.
It would be good to cleanup the unused imports and line lengths.
> 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
>
>
> 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]
