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