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

Hitesh Shah commented on TEZ-1216:
----------------------------------

[~bikassaha] Latest patch should be visible at 
https://github.com/apache/tez/pull/3/files

bq. We are optimistically hoping that unregister will be successful.
Not really. Having the check for 
this.taskSchedulerEventHandler.hasUnregistered() implies staging clean up only 
happens on successful unregister. 

bq. typo unregisterScheduleEventHandler()
Could you point out where you saw this? 

bq. shutdownHandlerRunning etc from the previous patch has disappeared?
Should still be part of the patch. 

bq.Could you please rename this to scratch dir or temp dir. This can be 
confusing.TEZ_AM_STAGING_DIR_AUTO_DELETE, its value and LOG "Deleting staging 
directory"
Addressed. 



> Clean up the staging directory when the application completes
> -------------------------------------------------------------
>
>                 Key: TEZ-1216
>                 URL: https://issues.apache.org/jira/browse/TEZ-1216
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Mohammad Kamrul Islam
>            Assignee: Hitesh Shah
>            Priority: Critical
>         Attachments: TEZ-1216.1.patch, TEZ-1216.2.patch
>
>
> This JIRA is a part of TEZ-693. It includes the removal of stage directory 
> when application's last attempt is done. In addition, it will check a config 
> to determine whether to auto clean it or not.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to