[jira] Commented: (MAPREDUCE-975) Add an API in job client to get the history file url for a given job id

["Jothi Padmanabhan (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MAPREDUCE-975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12757065#action_12757065 ] 

Jothi Padmanabhan commented on MAPREDUCE-975:
---------------------------------------------

Looks good. Minor nits

# JobTracker.getJobHistoryDir does not need to throw the exceptions
# It would be good to add an assert for null at the end of getDoneFile

> Add an API in job client to get the history file url for a given job id
> -----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-975
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-975
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: client, jobtracker
>            Reporter: Sharad Agarwal
>            Assignee: Sharad Agarwal
>             Fix For: 0.21.0
>
>         Attachments: 975_v1.patch, 975_v2.patch
>
>
> MAPREDUCE-817 added an API to get history url in RunningJob. Similar API should be added in job client to get the history file given a job id. Something like:
> String getHistoryFile(JobId jobid);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.