[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856428#comment-15856428
]
Jonathan Hung commented on YARN-6152:
-
Great, thanks [~eepayne] and [~sunilg]!
> Used
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856365#comment-15856365
]
Eric Payne commented on YARN-6152:
--
The patch LGTM. +1
The unit test failures are unrelat
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856203#comment-15856203
]
Eric Payne commented on YARN-6152:
--
Thanks [~jhung], I will review today.
> Used queue pe
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855696#comment-15855696
]
Sunil G commented on YARN-6152:
---
I am fine with latest patch. Will wait for review from [~eep
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855447#comment-15855447
]
Hadoop QA commented on YARN-6152:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855239#comment-15855239
]
Jonathan Hung commented on YARN-6152:
-
[~sunilg], thanks for the review. Uploaded 002 p
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855229#comment-15855229
]
Hadoop QA commented on YARN-6152:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855161#comment-15855161
]
Sunil G commented on YARN-6152:
---
Yes. We should use {{nlm.getResourceByLabel(partition, clust
[
https://issues.apache.org/jira/browse/YARN-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15855033#comment-15855033
]
Jonathan Hung commented on YARN-6152:
-
[~eepayne], do you mind taking a look at this pa