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

Jonathan Eagles updated TEZ-2787:
---------------------------------
    Attachment: TEZ-2787.2.patch

Added the tmpdir as part of constructAMLaunchOpts at the beginning so that 
default opts could override the tmpdir setting. Debated between this option and 
modifying the TEZ_AM_LAUNCH_CLUSTER_DEFAULT_CMD_OPTS_DEFAULT setting to contain 
the java.io.tmpdir setting instead.

> Tez AM should have java.io.tmpdir=./tmp to be consistent with tasks
> -------------------------------------------------------------------
>
>                 Key: TEZ-2787
>                 URL: https://issues.apache.org/jira/browse/TEZ-2787
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.7.0
>            Reporter: Jason Lowe
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-2787.1.patch, TEZ-2787.2.patch
>
>
> TezRuntimeChildJVM ensures that tasks are launched with 
> -Djava.io.tmpdir=./tmp, but there's no corresponding code to ensure the Tez 
> AM also has a similar tmpdir setting.  The client should setup the AM launch 
> context to have -Djava.io.tmpdir=./tmp to be consistent with the tasks and to 
> prevent accidental leaking of files in /tmp by the Tez AM if it crashes.



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

Reply via email to