From 71566e23820d33e0110ca55eded3299735e970b9 Mon Sep 17 00:00:00 2001 From: yliu Date: Tue, 18 Aug 2015 09:23:06 +0800 Subject: [PATCH] HDFS-8862. BlockManager#excessReplicateMap should use a HashMap. (yliu) --- hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 2 ++ .../hadoop/hdfs/server/blockmanagement/BlockManager.java | 3 +-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index d1b04dcb2f..132adc127e 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -820,6 +820,8 @@ Release 2.8.0 - UNRELEASED HDFS-8845. DiskChecker should not traverse the entire tree (Chang Li via Colin P. McCabe) + HDFS-8862. BlockManager#excessReplicateMap should use a HashMap. (yliu) + BUG FIXES HDFS-7501. TransactionsSinceLastCheckpoint can be negative on SBNs. diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java index cde6588a2b..aad7fec41f 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java @@ -33,7 +33,6 @@ import java.util.Map; import java.util.Queue; import java.util.Set; -import java.util.TreeMap; import java.util.TreeSet; import java.util.concurrent.ThreadLocalRandom; import java.util.concurrent.atomic.AtomicLong; @@ -204,7 +203,7 @@ public int getPendingDataNodeMessageCount() { * DataNode. We'll eventually remove these extras. */ public final Map> excessReplicateMap = - new TreeMap>(); + new HashMap<>(); /** * Store set of Blocks that need to be replicated 1 or more times.