From bf82a8de300bc7a511929277c2c9afd46de78c9e Mon Sep 17 00:00:00 2001 From: Tsz-Wo Nicholas Sze Date: Thu, 19 Nov 2015 14:54:01 -0800 Subject: [PATCH] HDFS-9434. Recommission a datanode with 500k blocks may pause NN for 30 seconds for printing info log messags. --- hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++ .../hadoop/hdfs/server/blockmanagement/BlockManager.java | 8 +++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index 64ed8937515..d4a1d132235 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -1474,6 +1474,9 @@ Release 2.8.0 - UNRELEASED HDFS-9400. TestRollingUpgradeRollback fails on branch-2. (Brahma Reddy Battula via cnauroth) + HDFS-9434. Recommission a datanode with 500k blocks may pause NN for 30 + seconds for printing info log messags. (szetszwo) + Release 2.7.3 - UNRELEASED INCOMPATIBLE CHANGES 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 e6b65b4f0bb..8efa02f1e70 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 @@ -2926,11 +2926,9 @@ public class BlockManager implements BlockStatsMXBean { for(DatanodeStorageInfo storage : blocksMap.getStorages(block, State.NORMAL)) { final DatanodeDescriptor cur = storage.getDatanodeDescriptor(); if (storage.areBlockContentsStale()) { - LOG.info("BLOCK* processOverReplicatedBlock: " + - "Postponing processing of over-replicated " + - block + " since storage + " + storage - + "datanode " + cur + " does not yet have up-to-date " + - "block information."); + LOG.trace("BLOCK* processOverReplicatedBlock: Postponing {}" + + " since storage {} does not yet have up-to-date information.", + block, storage); postponeBlock(block); return; }