[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17134338#comment-17134338
]
Wangda Tan commented on YARN-10293:
---
Missed last comments, thanks [~prabhujoseph]/[~Tao Yang]!
>
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17134262#comment-17134262
]
Hudson commented on YARN-10293:
---
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #18347 (See
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17134257#comment-17134257
]
Prabhu Joseph commented on YARN-10293:
--
Thanks [~Tao Yang] and [~wangda].
> Reserved Containers not
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133848#comment-17133848
]
Tao Yang commented on YARN-10293:
-
I think this patch is fine enough, and would like to commit the latest
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17132565#comment-17132565
]
Prabhu Joseph commented on YARN-10293:
--
Thanks [~Tao Yang] for the review.
[~wangda] Let me know if
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17129091#comment-17129091
]
Tao Yang commented on YARN-10293:
-
Thanks [~prabhujoseph] for updating the patch.
LGTM now, [~wangda], do
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17128340#comment-17128340
]
Hadoop QA commented on YARN-10293:
--
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17128183#comment-17128183
]
Prabhu Joseph commented on YARN-10293:
--
[~Tao Yang] Have used SchedulerNode#updateTotalResource to
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17127867#comment-17127867
]
Tao Yang commented on YARN-10293:
-
Thanks [~prabhujoseph] for updating the patch.
Another concern in UT
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17126771#comment-17126771
]
Hadoop QA commented on YARN-10293:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17126655#comment-17126655
]
Prabhu Joseph commented on YARN-10293:
--
Thanks [~Tao Yang] for the review. Have addressed below
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17126407#comment-17126407
]
Tao Yang commented on YARN-10293:
-
Thanks [~prabhujoseph] for this effort. I'm fine, please go ahead.
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17124807#comment-17124807
]
Prabhu Joseph commented on YARN-10293:
--
Thanks [~Tao Yang] for the comments. If you are fine, will
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17124527#comment-17124527
]
Tao Yang commented on YARN-10293:
-
Thanks [~wangda] for your confirmation.
I think the proposed change
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17124195#comment-17124195
]
Wangda Tan commented on YARN-10293:
---
[~Tao Yang], the suggestion totally make sense to me. When we have
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17123839#comment-17123839
]
Hadoop QA commented on YARN-10293:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17123686#comment-17123686
]
Tao Yang commented on YARN-10293:
-
Hi, [~prabhujoseph], [~wangda]
This problem is similar to YARN-9598,
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17123652#comment-17123652
]
Prabhu Joseph commented on YARN-10293:
--
Have attached a patch with removing the if condition. Will
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17121419#comment-17121419
]
Wangda Tan commented on YARN-10293:
---
[~prabhujoseph], I agree with you, I think the entire {{if}} check
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120982#comment-17120982
]
Prabhu Joseph commented on YARN-10293:
--
Thanks [~wangda] for reviewing.
The older behavior of
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17119866#comment-17119866
]
Wangda Tan commented on YARN-10293:
---
[~prabhujoseph],
This looks like a valid bug, but I'm wondering
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17119618#comment-17119618
]
Prabhu Joseph commented on YARN-10293:
--
[~ztang] [~leftnoteasy] Can you review this Jira when you
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17119616#comment-17119616
]
Hadoop QA commented on YARN-10293:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-10293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17118750#comment-17118750
]
Hadoop QA commented on YARN-10293:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
24 matches
Mail list logo