[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

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

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

2017-07-07 Thread Yeliang Cang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16077683#comment-16077683 ] Yeliang Cang commented on YARN-6765: Sorry about that! [~templedf], I have reverted those changes and

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

2017-07-06 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16076631#comment-16076631 ] Daniel Templeton commented on YARN-6765: Thanks for the patch, [~Cyl] and for being proactive about

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

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

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

2017-07-06 Thread Yeliang Cang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16076432#comment-16076432 ] Yeliang Cang commented on YARN-6765: patch002, fix one checkstyle and three findbug warnings. Two

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

2017-07-06 Thread Yeliang Cang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16076375#comment-16076375 ] Yeliang Cang commented on YARN-6765: Will fix findbugs and checkstyle warnings later! >

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

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

[jira] [Commented] (YARN-6765) CGroupsHandlerImpl.initializeControllerPaths() should include cause when chaining exceptions

2017-07-06 Thread Yeliang Cang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16076218#comment-16076218 ] Yeliang Cang commented on YARN-6765: Hi, [~templedf], I have submit a patch. Would you have a look?