You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Wei Yan (JIRA)" <ji...@apache.org> on 2014/06/12 18:50:01 UTC
[jira] [Updated] (YARN-2155) FairScheduler: Incorrect check when
trigger a preemption
[ https://issues.apache.org/jira/browse/YARN-2155?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wei Yan updated YARN-2155:
--------------------------
Description:
{code}
private boolean shouldAttemptPreemption() {
if (preemptionEnabled) {
return (preemptionUtilizationThreshold < Math.max(
(float) rootMetrics.getAvailableMB() / clusterResource.getMemory(),
(float) rootMetrics.getAvailableVirtualCores() /
clusterResource.getVirtualCores()));
}
return false;
}
{code}
preemptionUtilizationThreshould should be compared with allocatedResource instead of availableResource.
was:
{code}
private boolean shouldAttemptPreemption() {
if (preemptionEnabled) {
return (preemptionUtilizationThreshold < Math.max(
(float) rootMetrics.getAvailableMB() / clusterResource.getMemory(),
(float) rootMetrics.getAvailableVirtualCores() /
clusterResource.getVirtualCores()));
}
return false;
}
{code}
preemptionUtilizationThreshould should be compared with allocatedResource instead of availableResource.
> FairScheduler: Incorrect check when trigger a preemption
> --------------------------------------------------------
>
> Key: YARN-2155
> URL: https://issues.apache.org/jira/browse/YARN-2155
> Project: Hadoop YARN
> Issue Type: Bug
> Reporter: Wei Yan
> Assignee: Wei Yan
>
> {code}
> private boolean shouldAttemptPreemption() {
> if (preemptionEnabled) {
> return (preemptionUtilizationThreshold < Math.max(
> (float) rootMetrics.getAvailableMB() / clusterResource.getMemory(),
> (float) rootMetrics.getAvailableVirtualCores() /
> clusterResource.getVirtualCores()));
> }
> return false;
> }
> {code}
> preemptionUtilizationThreshould should be compared with allocatedResource instead of availableResource.
--
This message was sent by Atlassian JIRA
(v6.2#6252)