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

Hitesh Shah commented on TEZ-2787:
----------------------------------

+1 to [~jlowe]'s comment. This could be done by adding the code changes into 
the constructAMLaunchOpts() function before the user configs are applied. Would 
also make it easier to unit test this fix.  

> 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
>
>
> 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