[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-07 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15729545#comment-15729545 ] Sunil G commented on YARN-5965: --- Test case failure looks not related. Committing the patch. > Revisit

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

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

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-07 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15729280#comment-15729280 ] Sunil G commented on YARN-5965: --- Still jenkins has not ran on latest patch. I triggered a build manually now.

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-07 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15728059#comment-15728059 ] Sunil G commented on YARN-5965: --- +1 from my end as well. i could commit the same in a day if there are no

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15728033#comment-15728033 ] Jian He commented on YARN-5965: --- lgtm > Revisit ApplicationReport #getApplicationTimeouts >

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

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

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-06 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15725677#comment-15725677 ] Sunil G commented on YARN-5965: --- Thanks [~rohithsharma] Few minor nits: {noformat} optional

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15723482#comment-15723482 ] Jian He commented on YARN-5965: --- yep, sounds good to me. > Revisit ApplicationReport #getApplicationTimeouts

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-05 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721642#comment-15721642 ] Sunil G commented on YARN-5965: --- Hi. Currently we use map in REST end and for cli. So I think we can keep

[jira] [Commented] (YARN-5965) Revisit ApplicationReport #getApplicationTimeouts

2016-12-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721574#comment-15721574 ] Jian He commented on YARN-5965: --- [~sunilg], [~rohithsharma] your opinion ? I was working on YARN-5768,