Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-20052 TestRegionOpen#testNonExistentRegionReplica fails due to NPE

This commit is contained in:
tedyu 2018-02-22 12:41:13 -08:00
parent cadc2efd12
commit 4d9589b165
2 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionFileSystem.java
View File

@ -997,12 +997,11 @@ public class HRegionFileSystem {
// Write HRI to a file in case we need to recover hbase:meta // Write HRI to a file in case we need to recover hbase:meta
regionFs.writeRegionInfoOnFilesystem(false); regionFs.writeRegionInfoOnFilesystem(false);
return regionFs;
} else { } else {
if (LOG.isDebugEnabled()) if (LOG.isDebugEnabled())
LOG.debug("Skipping creation of .regioninfo file for " + regionInfo); LOG.debug("Skipping creation of .regioninfo file for " + regionInfo);
} }
return null; return regionFs;
} }
/** /**

2
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionOpen.java
View File

@ -47,7 +47,6 @@ import org.apache.hadoop.hbase.util.Bytes;
import org.apache.hadoop.hbase.util.FSUtils; import org.apache.hadoop.hbase.util.FSUtils;
import org.junit.AfterClass; import org.junit.AfterClass;
import org.junit.BeforeClass; import org.junit.BeforeClass;
import org.junit.Ignore;
import org.junit.Rule; import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.experimental.categories.Category; import org.junit.experimental.categories.Category;
@ -98,7 +97,6 @@ public class TestRegionOpen {
assertEquals(2, exec.getCompletedTaskCount()); assertEquals(2, exec.getCompletedTaskCount());
} }
@Ignore // Needs rewrite since HBASE-19391 which returns null out of createRegionOnFileSystem
@Test @Test
public void testNonExistentRegionReplica() throws Exception { public void testNonExistentRegionReplica() throws Exception {
final TableName tableName = TableName.valueOf(name.getMethodName()); final TableName tableName = TableName.valueOf(name.getMethodName());