You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by jb...@apache.org on 2021/05/25 18:08:27 UTC
[hadoop] branch trunk updated: YARN-10691.
DominantResourceCalculator isInvalidDivisor should consider only countable
resource types. Contributed by Bilwa S T.
This is an automated email from the ASF dual-hosted git repository.
jbrennan pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/trunk by this push:
new c458182 YARN-10691. DominantResourceCalculator isInvalidDivisor should consider only countable resource types. Contributed by Bilwa S T.
c458182 is described below
commit c4581827a92d9715ad1f11eb4012e4d537bb4046
Author: Jim Brennan <jb...@apache.org>
AuthorDate: Tue May 25 18:07:28 2021 +0000
YARN-10691. DominantResourceCalculator isInvalidDivisor should consider only countable resource types. Contributed by Bilwa S T.
---
.../apache/hadoop/yarn/util/resource/DominantResourceCalculator.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/resource/DominantResourceCalculator.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/resource/DominantResourceCalculator.java
index db62a89..135f348 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/resource/DominantResourceCalculator.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/util/resource/DominantResourceCalculator.java
@@ -380,8 +380,9 @@ public class DominantResourceCalculator extends ResourceCalculator {
@Override
public boolean isInvalidDivisor(Resource r) {
- for (ResourceInformation res : r.getResources()) {
- if (res.getValue() == 0L) {
+ int maxLength = ResourceUtils.getNumberOfCountableResourceTypes();
+ for (int i = 0; i < maxLength; i++) {
+ if (r.getResourceInformation(i).getValue() == 0L) {
return true;
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org