[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-28 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15615799#comment-15615799 ] Hudson commented on YARN-5027: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10720 (See

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613337#comment-15613337 ] Hadoop QA commented on YARN-5027: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Chris Trezzo (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613303#comment-15613303 ] Chris Trezzo commented on YARN-5027: bq. The exists check is on the root log dir, not the app log dirs.

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613298#comment-15613298 ] Jason Lowe commented on YARN-5027: -- +1 for the latest patch. I'll commit this tomorrow if there are no

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613285#comment-15613285 ] Jason Lowe commented on YARN-5027: -- I don't believe it will leak those directories. The exists check is

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613265#comment-15613265 ] Hadoop QA commented on YARN-5027: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Chris Trezzo (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613251#comment-15613251 ] Chris Trezzo commented on YARN-5027: Does the current patch have the potential to leak

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-10-27 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15613182#comment-15613182 ] Jason Lowe commented on YARN-5027: -- Thanks for the patch! Looks good to me, kicking another Jenkins run

[jira] [Commented] (YARN-5027) NM should clean up app log dirs after NM restart

2016-05-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15283512#comment-15283512 ] Hadoop QA commented on YARN-5027: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||