diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt index 7cdb036281..9e49df7e67 100644 --- a/hadoop-yarn-project/CHANGES.txt +++ b/hadoop-yarn-project/CHANGES.txt @@ -178,6 +178,9 @@ Release 2.2.1 - UNRELEASED YARN-1343. NodeManagers additions/restarts are not reported as node updates in AllocateResponse responses to AMs. (tucu) + YARN-1381. Same relaxLocality appears twice in exception message of + AMRMClientImpl#checkLocalityRelaxationConflict() (Ted Yu via Sandy Ryza) + Release 2.2.0 - 2013-10-13 INCOMPATIBLE CHANGES diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/AMRMClientImpl.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/AMRMClientImpl.java index 061c50bd7b..f7ed0ac314 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/AMRMClientImpl.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/AMRMClientImpl.java @@ -499,13 +499,15 @@ private void checkLocalityRelaxationConflict(Priority priority, for (String location : locations) { TreeMap reqs = remoteRequests.get(location); - if (reqs != null && !reqs.isEmpty() - && reqs.values().iterator().next().remoteRequest.getRelaxLocality() - != relaxLocality) { - throw new InvalidContainerRequestException("Cannot submit a " - + "ContainerRequest asking for location " + location - + " with locality relaxation " + relaxLocality + " when it has " - + "already been requested with locality relaxation " + relaxLocality); + if (reqs != null && !reqs.isEmpty()) { + boolean existingRelaxLocality = + reqs.values().iterator().next().remoteRequest.getRelaxLocality(); + if (relaxLocality != existingRelaxLocality) { + throw new InvalidContainerRequestException("Cannot submit a " + + "ContainerRequest asking for location " + location + + " with locality relaxation " + relaxLocality + " when it has " + + "already been requested with locality relaxation " + existingRelaxLocality); + } } } }