Author: brandonli
Date: Wed Dec 18 23:29:31 2013
New Revision: 1552163
URL: http://svn.apache.org/r1552163
Log:
HDFS-5662. Merging change r1552135 from branch-2
Modified:
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDecommission.java
Modified:
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
URL:
http://svn.apache.org/viewvc/hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt?rev=1552163&r1=1552162&r2=1552163&view=diff
==============================================================================
---
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
(original)
+++
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
Wed Dec 18 23:29:31 2013
@@ -28,6 +28,9 @@ Release 2.3.0 - UNRELEASED
HDFS-4983. Numeric usernames do not work with WebHDFS FS. (Yongjun Zhang
via
jing9)
+ HDFS-5662. Can't decommission a DataNode due to file's replication factor
+ larger than the rest of the cluster size. (brandonli)
+
OPTIMIZATIONS
BUG FIXES
Modified:
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java
URL:
http://svn.apache.org/viewvc/hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java?rev=1552163&r1=1552162&r2=1552163&view=diff
==============================================================================
---
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java
(original)
+++
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockManager.java
Wed Dec 18 23:29:31 2013
@@ -2854,6 +2854,7 @@ assert storedBlock.findDatanode(dn) < 0
*/
boolean isReplicationInProgress(DatanodeDescriptor srcNode) {
boolean status = false;
+ boolean firstReplicationLog = true;
int underReplicatedBlocks = 0;
int decommissionOnlyReplicas = 0;
int underReplicatedInOpenFiles = 0;
@@ -2868,10 +2869,17 @@ assert storedBlock.findDatanode(dn) < 0
int curExpectedReplicas = getReplication(block);
if (isNeededReplication(block, curExpectedReplicas, curReplicas)) {
if (curExpectedReplicas > curReplicas) {
- //Log info about one block for this node which needs replication
+ // Log info about one block for this node which needs replication
if (!status) {
status = true;
- logBlockReplicationInfo(block, srcNode, num);
+ if (firstReplicationLog) {
+ logBlockReplicationInfo(block, srcNode, num);
+ }
+ // Allowing decommission as long as default replication is met
+ if (curReplicas >= defaultReplication) {
+ status = false;
+ firstReplicationLog = false;
+ }
}
underReplicatedBlocks++;
if ((curReplicas == 0) && (num.decommissionedReplicas() > 0)) {
Modified:
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDecommission.java
URL:
http://svn.apache.org/viewvc/hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDecommission.java?rev=1552163&r1=1552162&r2=1552163&view=diff
==============================================================================
---
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDecommission.java
(original)
+++
hadoop/common/branches/branch-2.3/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDecommission.java
Wed Dec 18 23:29:31 2013
@@ -43,7 +43,6 @@ import org.apache.hadoop.hdfs.protocol.D
import org.apache.hadoop.hdfs.protocol.HdfsConstants.DatanodeReportType;
import org.apache.hadoop.hdfs.protocol.LocatedBlock;
import org.apache.hadoop.hdfs.server.namenode.FSNamesystem;
-import org.apache.hadoop.hdfs.server.namenode.HostFileManager;
import org.apache.hadoop.hdfs.server.namenode.NameNode;
import org.apache.hadoop.hdfs.server.namenode.NameNodeAdapter;
import org.junit.After;
@@ -337,6 +336,51 @@ public class TestDecommission {
}
/**
+ * Tests decommission with replicas on the target datanode cannot be migrated
+ * to other datanodes and satisfy the replication factor. Make sure the
+ * datanode won't get stuck in decommissioning state.
+ */
+ @Test(timeout = 360000)
+ public void testDecommission2() throws IOException {
+ LOG.info("Starting test testDecommission");
+ int numNamenodes = 1;
+ int numDatanodes = 4;
+ conf.setInt(DFSConfigKeys.DFS_REPLICATION_KEY, 3);
+ startCluster(numNamenodes, numDatanodes, conf);
+
+ ArrayList<ArrayList<DatanodeInfo>> namenodeDecomList = new
ArrayList<ArrayList<DatanodeInfo>>(
+ numNamenodes);
+ namenodeDecomList.add(0, new ArrayList<DatanodeInfo>(numDatanodes));
+
+ Path file1 = new Path("testDecommission2.dat");
+ int replicas = 4;
+
+ // Start decommissioning one namenode at a time
+ ArrayList<DatanodeInfo> decommissionedNodes = namenodeDecomList.get(0);
+ FileSystem fileSys = cluster.getFileSystem(0);
+ writeFile(fileSys, file1, replicas);
+
+ // Decommission one node. Verify that node is decommissioned.
+ DatanodeInfo decomNode = decommissionNode(0, decommissionedNodes,
+ AdminStates.DECOMMISSIONED);
+ decommissionedNodes.add(decomNode);
+
+ // Ensure decommissioned datanode is not automatically shutdown
+ DFSClient client = getDfsClient(cluster.getNameNode(0), conf);
+ assertEquals("All datanodes must be alive", numDatanodes,
+ client.datanodeReport(DatanodeReportType.LIVE).length);
+ assertNull(checkFile(fileSys, file1, replicas, decomNode.getXferAddr(),
+ numDatanodes));
+ cleanupFile(fileSys, file1);
+
+ // Restart the cluster and ensure recommissioned datanodes
+ // are allowed to register with the namenode
+ cluster.shutdown();
+ startCluster(1, 4, conf);
+ cluster.shutdown();
+ }
+
+ /**
* Tests recommission for non federated cluster
*/
@Test(timeout=360000)
@@ -381,7 +425,20 @@ public class TestDecommission {
DFSClient client = getDfsClient(cluster.getNameNode(i), conf);
assertEquals("All datanodes must be alive", numDatanodes,
client.datanodeReport(DatanodeReportType.LIVE).length);
- assertNull(checkFile(fileSys, file1, replicas,
decomNode.getXferAddr(), numDatanodes));
+ // wait for the block to be replicated
+ int tries = 0;
+ while (tries++ < 20) {
+ try {
+ Thread.sleep(1000);
+ if (checkFile(fileSys, file1, replicas, decomNode.getXferAddr(),
+ numDatanodes) == null) {
+ break;
+ }
+ } catch (InterruptedException ie) {
+ }
+ }
+ assertTrue("Checked if block was replicated after decommission, tried "
+ + tries + " times.", tries < 20);
cleanupFile(fileSys, file1);
}
}
@@ -422,12 +479,25 @@ public class TestDecommission {
DFSClient client = getDfsClient(cluster.getNameNode(i), conf);
assertEquals("All datanodes must be alive", numDatanodes,
client.datanodeReport(DatanodeReportType.LIVE).length);
- assertNull(checkFile(fileSys, file1, replicas, decomNode.getXferAddr(),
numDatanodes));
+ int tries =0;
+ // wait for the block to be replicated
+ while (tries++ < 20) {
+ try {
+ Thread.sleep(1000);
+ if (checkFile(fileSys, file1, replicas, decomNode.getXferAddr(),
+ numDatanodes) == null) {
+ break;
+ }
+ } catch (InterruptedException ie) {
+ }
+ }
+ assertTrue("Checked if block was replicated after decommission, tried "
+ + tries + " times.", tries < 20);
// stop decommission and check if the new replicas are removed
recomissionNode(decomNode);
// wait for the block to be deleted
- int tries = 0;
+ tries = 0;
while (tries++ < 20) {
try {
Thread.sleep(1000);
