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

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

Code change for the fix looks good. However a minor nit - unit test will likely 
fail on Windows. 

> 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