[jira] [Commented] (MAPREDUCE-6521) MiniMRYarnCluster should not create /tmp/hadoop-yarn/staging on local filesystem in unit test

["Masatake Iwasaki (JIRA)" <ji...@apache.org>]

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

Masatake Iwasaki commented on MAPREDUCE-6521:
---------------------------------------------

{code}
    if (conf.get(MRJobConfig.MR_AM_STAGING_DIR) == null) {
      conf.set(MRJobConfig.MR_AM_STAGING_DIR, new File(getTestWorkDir(),
          "apps_staging_dir/").getAbsolutePath());
    }
{code}
The intent of the if statement seems to be "if yarn.app.mapreduce.am.staging-dir is not specified in configuration" but the {{conf.get(MRJobConfig.MR_AM_STAGING_DIR)}} always returns non-null value because the default value is defined in mapred-default.xml as {{/tmp/hadoop-yarn/staging}}.

> MiniMRYarnCluster should not create /tmp/hadoop-yarn/staging on local filesystem in unit test
> ---------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6521
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6521
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: test
>            Reporter: Masatake Iwasaki
>            Assignee: Masatake Iwasaki
>
> MiniMRYarnCluster create /tmp/hadoop-yarn/staging/history/done by default. It should be under {{testWorkDir}} if the file system is localFs in order to make it to be removed by {{mvn clean}}. It would also avoid issues under parallel unit testing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)