YARN-4690. Skip object allocation in FSAppAttempt#getResourceUsage when possible (Ming Ma via sjlee)

(cherry picked from commit 7de70680fe)
This commit is contained in:
Sangjin Lee 2016-02-17 20:55:21 -08:00
parent a4492dc6ac
commit 6475d468f5
2 changed files with 9 additions and 1 deletions

View File

@ -964,6 +964,9 @@ Release 2.6.5 - UNRELEASED
OPTIMIZATIONS
YARN-4690. Skip object allocation in FSAppAttempt#getResourceUsage when
possible (Ming Ma via sjlee)
BUG FIXES
Release 2.6.4 - 2016-02-11

View File

@ -741,7 +741,12 @@ public class FSAppAttempt extends SchedulerApplicationAttempt
public Resource getResourceUsage() {
// Here the getPreemptedResources() always return zero, except in
// a preemption round
return Resources.subtract(getCurrentConsumption(), getPreemptedResources());
// In the common case where preempted resource is zero, return the
// current consumption Resource object directly without calling
// Resources.subtract which creates a new Resource object for each call.
return getPreemptedResources().equals(Resources.none()) ?
getCurrentConsumption() :
Resources.subtract(getCurrentConsumption(), getPreemptedResources());
}
@Override