YARN-6984. DominantResourceCalculator.isAnyMajorResourceZero() should test all resources

(Contributed by Sunil G via Daniel Templeton)
This commit is contained in:
Daniel Templeton 2017-10-24 10:04:36 -07:00
parent b90750cfb8
commit 679f99b145

View File

@ -557,6 +557,14 @@ public boolean fitsIn(Resource smaller, Resource bigger) {
@Override
public boolean isAnyMajorResourceZero(Resource resource) {
return resource.getMemorySize() == 0f || resource.getVirtualCores() == 0;
int maxLength = ResourceUtils.getNumberOfKnownResourceTypes();
for (int i = 0; i < maxLength; i++) {
ResourceInformation resourceInformation = resource
.getResourceInformation(i);
if (resourceInformation.getValue() == 0L) {
return true;
}
}
return false;
}
}