YARN-10997. Revisit allocation and reservation logging. Contributed by Andras Gyori
(cherry picked from commit 7cb887e6c2
)
This commit is contained in:
parent
690337bca9
commit
ebe3ed5cd3
|
@ -3131,13 +3131,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");
|
||||
}
|
||||
|
||||
LOG.debug("Allocation proposal accepted={}, proposal={}", isSuccess,
|
||||
|
|
|
@ -357,6 +357,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 =
|
||||
|
@ -572,6 +573,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;
|
||||
}
|
||||
|
||||
|
@ -670,12 +673,14 @@ public class FiCaSchedulerApp extends SchedulerApplicationAttempt {
|
|||
schedulerContainer.getRmContainer().getContainer(),
|
||||
reReservation);
|
||||
|
||||
LOG.info("Reserved container=" + rmContainer.getContainerId()
|
||||
if (LOG.isDebugEnabled()) {
|
||||
LOG.debug("Reserved container=" + rmContainer.getContainerId()
|
||||
+ ", on node=" + schedulerContainer.getSchedulerNode()
|
||||
+ " with resource=" + rmContainer
|
||||
.getAllocatedOrReservedResource());
|
||||
}
|
||||
}
|
||||
}
|
||||
} finally {
|
||||
writeLock.unlock();
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue