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

HBASE-24481 REST - Fix incorrect response code of get-regions in rest api 

Closes #2425

Signed-off-by: Viraj Jasani <vjasani@apache.org>
This commit is contained in:
KevinSmile 2020-09-19 19:45:10 +05:30 committed by Viraj Jasani
parent de2f538bc9
commit 7aa36ae10a
No known key found for this signature in database
GPG Key ID: B3D6C0B41C8ADFD5
2 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RegionsResource.java
View File

@ -75,6 +75,9 @@ public class RegionsResource extends ResourceBase {
servlet.getMetrics().incrementRequests(1); servlet.getMetrics().incrementRequests(1);
try { try {
TableName tableName = TableName.valueOf(tableResource.getName()); TableName tableName = TableName.valueOf(tableResource.getName());
if (!tableResource.exists()) {
throw new TableNotFoundException(tableName);
}
TableInfoModel model = new TableInfoModel(tableName.getNameAsString()); TableInfoModel model = new TableInfoModel(tableName.getNameAsString());
List<HRegionLocation> locs; List<HRegionLocation> locs;

9
hbase-rest/src/test/java/org/apache/hadoop/hbase/rest/TestTableResource.java
View File

@ -261,5 +261,14 @@ public class TestTableResource {
checkTableInfo(model); checkTableInfo(model);
} }
@Test
public void testTableNotFound() throws IOException {
String notExistTable = "notexist";
Response response1 = client.get("/" + notExistTable + "/schema", Constants.MIMETYPE_JSON);
assertEquals(404, response1.getCode());
Response response2 = client.get("/" + notExistTable + "/regions", Constants.MIMETYPE_XML);
assertEquals(404, response2.getCode());
}
} }