You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by te...@apache.org on 2017/04/05 23:07:31 UTC
hadoop git commit: YARN-6381. FSAppAttempt has several variables that
should be final (Contributed by Ameet Zaveri via Daniel Templeton)
Repository: hadoop
Updated Branches:
refs/heads/trunk 539ef5aa2 -> a2c57bb70
YARN-6381. FSAppAttempt has several variables that should be final
(Contributed by Ameet Zaveri via Daniel Templeton)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/a2c57bb7
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/a2c57bb7
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/a2c57bb7
Branch: refs/heads/trunk
Commit: a2c57bb70d719f72d59413f33ae67781b583b9f8
Parents: 539ef5a
Author: Daniel Templeton <te...@apache.org>
Authored: Wed Apr 5 16:06:00 2017 -0700
Committer: Daniel Templeton <te...@apache.org>
Committed: Wed Apr 5 16:07:24 2017 -0700
----------------------------------------------------------------------
.../resourcemanager/scheduler/fair/FSAppAttempt.java | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/a2c57bb7/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
----------------------------------------------------------------------
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
index ccfcffb..e0dfb73 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
@@ -74,11 +74,11 @@ public class FSAppAttempt extends SchedulerApplicationAttempt
private static final DefaultResourceCalculator RESOURCE_CALCULATOR
= new DefaultResourceCalculator();
- private long startTime;
- private Priority appPriority;
- private ResourceWeights resourceWeights;
+ private final long startTime;
+ private final Priority appPriority;
+ private final ResourceWeights resourceWeights;
private Resource demand = Resources.createResource(0);
- private FairScheduler scheduler;
+ private final FairScheduler scheduler;
private FSQueue fsQueue;
private Resource fairShare = Resources.createResource(0, 0);
@@ -96,9 +96,9 @@ public class FSAppAttempt extends SchedulerApplicationAttempt
// Used to record node reservation by an app.
// Key = RackName, Value = Set of Nodes reserved by app on rack
- private Map<String, Set<String>> reservations = new HashMap<>();
+ private final Map<String, Set<String>> reservations = new HashMap<>();
- private List<FSSchedulerNode> blacklistNodeIds = new ArrayList<>();
+ private final List<FSSchedulerNode> blacklistNodeIds = new ArrayList<>();
/**
* Delay scheduling: We often want to prioritize scheduling of node-local
* containers over rack-local or off-switch containers. To achieve this
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org