[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-12-05 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278523#comment-16278523 ] Hudson commented on YARN-7586: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13324 (See

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-12-05 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278503#comment-16278503 ] Sunil G commented on YARN-7586: --- Thanks [~suma.shivaprasad]. Thanks [~leftnoteasy] for comments. Committed to

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-12-05 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278238#comment-16278238 ] Sunil G commented on YARN-7586: --- Test case failures are not related. Committing shortly if there are no

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

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

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-12-01 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16274342#comment-16274342 ] Sunil G commented on YARN-7586: --- Thanks [~suma.shivaprasad]. Few comments # Do we need to make

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

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

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

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

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-11-30 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16273944#comment-16273944 ] Sunil G commented on YARN-7586: --- I think validateApplicationTimeouts could be placed in RMAppManager itself

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

2017-11-30 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16273604#comment-16273604 ] Wangda Tan commented on YARN-7586: -- Thanks [~suma.shivaprasad], change looks straightforward to me. One

[jira] [Commented] (YARN-7586) Application Placement should be done before ACL checks in ResourceManager

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