[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

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

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-10-22 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659605#comment-16659605 ] Eric Yang commented on YARN-8542: - [~csingh] Thank you for the patch. +1 for patch 002, will commit

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-10-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16657631#comment-16657631 ] Hadoop QA commented on YARN-8542: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-10-19 Thread Chandni Singh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16657575#comment-16657575 ] Chandni Singh commented on YARN-8542: - Addressed checkstyle & license warning in patch 2 > Yarn

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

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

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-10-19 Thread Chandni Singh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16657513#comment-16657513 ] Chandni Singh commented on YARN-8542: - [~gsaha] [~eyang] could you please review? > Yarn Service: Add

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-07-18 Thread Chandni Singh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16548379#comment-16548379 ] Chandni Singh commented on YARN-8542: - [~gsaha] Ok. That sounds reasonable. Will change it to the

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-07-18 Thread Gour Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16548375#comment-16548375 ] Gour Saha commented on YARN-8542: - [~csingh] agreed that the API is to request for containers. However,

[jira] [Commented] (YARN-8542) Yarn Service: Add component name to container json

2018-07-18 Thread Chandni Singh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16548248#comment-16548248 ] Chandni Singh commented on YARN-8542: - [~gsaha]  I am not in favor of the below format: {code:java} {