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

HDFS-14259. RBF: Fix safemode message for Router. Contributed by Ranith Sadar.

Browse Source
This commit is contained in:
Inigo Goiri 2019-03-02 17:17:34 -08:00 committed by Brahma Reddy Battula
parent 1c7ab59be3
commit 7bbe01a196
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/metrics/NamenodeBeanMetrics.java
View File

@ -234,7 +234,7 @@ public long getProvidedCapacity() {
@Override
public String getSafemode() {
try {
if (!getRouter().isRouterState(RouterServiceState.SAFEMODE)) {
if (getRouter().isRouterState(RouterServiceState.SAFEMODE)) {
return "Safe mode is ON. " + this.getSafeModeTip();
}
} catch (IOException e) {

8
hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java
View File

@ -747,6 +747,9 @@ public void testSafeModeStatus() throws Exception {
FederationMetrics metrics = router.getMetrics();
String jsonString = metrics.getRouterStatus();
String result = router.getNamenodeMetrics().getSafemode();
assertTrue("Wrong safe mode message: " + result,
result.startsWith("Safe mode is ON."));
// verify state using FederationMetrics
assertEquals(RouterServiceState.SAFEMODE.toString(), jsonString);
@ -756,6 +759,9 @@ public void testSafeModeStatus() throws Exception {
assertEquals(0,
ToolRunner.run(admin, new String[] {"-safemode", "leave" }));
jsonString = metrics.getRouterStatus();
result = router.getNamenodeMetrics().getSafemode();
assertEquals("Wrong safe mode message: " + result, "", result);
// verify state
assertEquals(RouterServiceState.RUNNING.toString(), jsonString);
assertFalse(routerContext.getRouter().getSafemodeService().isInSafeMode());
@ -1153,4 +1159,4 @@ public void testGetDestination() throws Exception {
argv = new String[] {"-getDestination /file1.txt /file2.txt"};
assertEquals(-1, ToolRunner.run(admin, argv));
}
}
}