You are viewing a plain text version of this content. The canonical link for it is here.

Posted to mapreduce-issues@hadoop.apache.org by "Sandy Ryza (JIRA)" <ji...@apache.org> on 2013/10/01 20:34:26 UTC

[jira] [Commented] (MAPREDUCE-5544) JobClient#getJob loads job conf twice

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

Sandy Ryza commented on MAPREDUCE-5544:
---------------------------------------

Per offline discussion with [~tucu], uploading a new patch that makes the new constructor private and makes the constructors delegate to each other.

> JobClient#getJob loads job conf twice
> -------------------------------------
>
>                 Key: MAPREDUCE-5544
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5544
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: MAPREDUCE-5544-1.patch, MAPREDUCE-5544.patch
>
>
> Calling JobClient#getJob causes the job conf file to be loaded twice, once in the constructor of JobClient.NetworkedJob and once in Cluster#getJob.  We should remove the former.
> MAPREDUCE-5001 was meant to fix a race that was causing problems in Hive tests, but the problem persists because it only fixed one of the places where the job conf file is loaded.



--
This message was sent by Atlassian JIRA
(v6.1#6144)