big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDFS-7154. Fix returning value of starting reconfiguration task (Lei Xu via Colin P. McCabe)

Browse Source
This commit is contained in:
Colin Patrick Mccabe 2014-10-20 18:33:11 -07:00
parent 4799570dfd
commit 7aab5fa1bd
3 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
View File

@ -1215,6 +1215,9 @@ Release 2.6.0 - UNRELEASED
HDFS-7167. NPE while running Mover if the given path is for a file. (jing9)
HDFS-7154. Fix returning value of starting reconfiguration task (Lei Xu via
Colin P. McCabe)
Release 2.5.1 - 2014-09-05
INCOMPATIBLE CHANGES

7
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java
View File

@ -1396,11 +1396,12 @@ int startReconfiguration(String nodeType, String address) throws IOException {
ClientDatanodeProtocol dnProxy = getDataNodeProxy(address);
dnProxy.startReconfiguration();
System.out.println("Started reconfiguration task on DataNode " + address);
return 0;
} else {
System.err.println("Node type " + nodeType +
" does not support reconfiguration.");
return 1;
}
return -1;
}
int getReconfigurationStatus(String nodeType, String address,
@ -1438,11 +1439,11 @@ int getReconfigurationStatus(String nodeType, String address,
}
} catch (IOException e) {
err.println("DataNode reloading configuration: " + e + ".");
return -1;
return 1;
}
} else {
err.println("Node type " + nodeType + " does not support reconfiguration.");
return -1;
return 1;
}
return 0;
}

3
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSAdmin.java
View File

@ -35,7 +35,6 @@
import java.io.IOException;
import java.io.PrintStream;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.Scanner;
@ -109,7 +108,7 @@ public void testGetReconfigureStatus()
final int port = datanode.getIpcPort();
final String address = "localhost:" + port;
admin.startReconfiguration("datanode", address);
assertThat(admin.startReconfiguration("datanode", address), is(0));
List<String> outputs = null;
int count = 100;