YARN-10997. Revisit allocation and reservation logging. Contributed by Andras Gyori

(cherry picked from commit 7cb887e6c2)
This commit is contained in:
Szilard Nemeth 2021-11-12 15:42:14 +01:00 committed by Eric Payne
parent cdbc2bac02
commit de35dd819a
2 changed files with 9 additions and 6 deletions

View File

@ -3074,13 +3074,11 @@ public class CapacityScheduler extends
long commitSuccess = System.nanoTime() - commitStart;
CapacitySchedulerMetrics.getMetrics()
.addCommitSuccess(commitSuccess);
LOG.info("Allocation proposal accepted");
isSuccess = true;
} else{
long commitFailed = System.nanoTime() - commitStart;
CapacitySchedulerMetrics.getMetrics()
.addCommitFailure(commitFailed);
LOG.info("Failed to accept allocation proposal");
}
if (LOG.isDebugEnabled()) {

View File

@ -358,6 +358,7 @@ public class FiCaSchedulerApp extends SchedulerApplicationAttempt {
// adding NP check as this proposal could not be allocated from reserved
// container in async-scheduling mode
if (allocation.getAllocateFromReservedContainer() == null) {
LOG.debug("Trying to allocate from reserved container in async scheduling mode");
return false;
}
RMContainer fromReservedContainer =
@ -576,6 +577,8 @@ public class FiCaSchedulerApp extends SchedulerApplicationAttempt {
if (updatePending &&
getOutstandingAsksCount(schedulerContainer.getSchedulerRequestKey())
<= 0) {
LOG.debug("Rejecting appliance of allocation due to existing pending allocation " +
"request for " + schedulerContainer);
return false;
}
@ -674,10 +677,12 @@ public class FiCaSchedulerApp extends SchedulerApplicationAttempt {
schedulerContainer.getRmContainer().getContainer(),
reReservation);
LOG.info("Reserved container=" + rmContainer.getContainerId()
+ ", on node=" + schedulerContainer.getSchedulerNode()
+ " with resource=" + rmContainer
.getAllocatedOrReservedResource());
if (LOG.isDebugEnabled()) {
LOG.debug("Reserved container=" + rmContainer.getContainerId()
+ ", on node=" + schedulerContainer.getSchedulerNode()
+ " with resource=" + rmContainer
.getAllocatedOrReservedResource());
}
}
}
} finally {