[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16991902#comment-16991902 ] Jonathan Hung commented on YARN-8179: - Seems this was committed to branch-2.10, adding 2.10.1 fix

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16975264#comment-16975264 ] Eric Payne commented on YARN-8179: -- I would like to backport this to 2.10 so that preemption works

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16483046#comment-16483046 ] Eric Payne commented on YARN-8179: -- Committed to trunk, branch-3.1, and branch-3.0. Thanks for the good

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16483035#comment-16483035 ] Hudson commented on YARN-8179: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14247 (See

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16482855#comment-16482855 ] Eric Payne commented on YARN-8179: -- {quote}Can we please change the {{newInstance}} call to

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16482841#comment-16482841 ] Eric Payne commented on YARN-8179: -- [~kyungwan nam], I am really sorry for the long delay, and I'm also

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16463041#comment-16463041 ] Eric Payne commented on YARN-8179: -- [~sunilg], if there is no objection, I'll commit this tomorrow. >

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16458804#comment-16458804 ] Eric Payne commented on YARN-8179: -- [~kyungwan nam], thanks for the new patch. +1 > Preemption does not

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

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

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16458432#comment-16458432 ] kyungwan nam commented on YARN-8179: Attached a new patch 003, which is based on up-to-date trunk. >

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

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

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16447705#comment-16447705 ] kyungwan nam commented on YARN-8179: Attached a patch with test case. and it will normalize up only if

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446001#comment-16446001 ] Eric Payne commented on YARN-8179: -- Can {{qT.toBePreempted}} ever be <0,0>? If so, I don't think it would

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16445532#comment-16445532 ] kyungwan nam commented on YARN-8179: Thanks [~eepayne], [~sunilg] for your comments. I'll work on new

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1638#comment-1638 ] Eric Payne commented on YARN-8179: -- bq. I want to make sure that it doesn't cause unnecessary preemption,

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16444098#comment-16444098 ] Eric Payne commented on YARN-8179: -- Thanks [~kyungwan nam] for raising the issue and providing a patch.

[jira] [Commented] (YARN-8179) Preemption does not happen due to natural_termination_factor when DRF is used

[ https://issues.apache.org/jira/browse/YARN-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16443654#comment-16443654 ] Sunil G commented on YARN-8179: --- [~kyungwan nam] Thanks for the patch Cud u pls write a test case to verify