diff --git a/hbase-rest/pom.xml b/hbase-rest/pom.xml index 6dd7bddc1b6..37aa174fd81 100644 --- a/hbase-rest/pom.xml +++ b/hbase-rest/pom.xml @@ -507,6 +507,18 @@ org.apache.hadoop hadoop-auth + + + com.sun.jersey + jersey-servlet + 1.19.4 + + + javax.ws.rs + jsr311-api + + + diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java index 3411ef28ab9..32605ae63f9 100644 --- a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java +++ b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java @@ -118,7 +118,7 @@ public class RESTServer implements Constants { HelpFormatter formatter = new HelpFormatter(); formatter.printHelp("hbase rest start", "", options, "\nTo run the REST server as a daemon, execute " + - "hbase-daemon.sh start|stop rest [--infoport ] [-p ] [-ro]\n", true); + "hbase-daemon.sh start|stop rest [-i ] [-p ] [-ro]\n", true); System.exit(exitCode); } @@ -186,7 +186,7 @@ public class RESTServer implements Constants { options.addOption("p", "port", true, "Port to bind to [default: " + DEFAULT_LISTEN_PORT + "]"); options.addOption("ro", "readonly", false, "Respond only to GET HTTP " + "method requests [default: false]"); - options.addOption(null, "infoport", true, "Port for web UI"); + options.addOption("i", "infoport", true, "Port for WEB UI"); CommandLine commandLine = null; try { @@ -218,7 +218,7 @@ public class RESTServer implements Constants { String val = commandLine.getOptionValue("infoport"); conf.setInt("hbase.rest.info.port", Integer.parseInt(val)); if (LOG.isDebugEnabled()) { - LOG.debug("Web UI port set to " + val); + LOG.debug("WEB UI port set to " + val); } } diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java index d4f7e9f69a1..4c4e7c45899 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java @@ -731,6 +731,7 @@ public class HBaseTestingUtility extends HBaseZKTestingUtility { // Frustrate yarn's and hdfs's attempts at writing /tmp. // Below is fragile. Make it so we just interpolate any 'tmp' reference. createDirAndSetProperty("yarn.node-labels.fs-store.root-dir"); + createDirAndSetProperty("yarn.node-attribute.fs-store.root-dir"); createDirAndSetProperty("yarn.nodemanager.log-dirs"); createDirAndSetProperty("yarn.nodemanager.remote-app-log-dir"); createDirAndSetProperty("yarn.timeline-service.entity-group-fs-store.active-dir"); diff --git a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestBindExceptionHandling.java b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestBindExceptionHandling.java index 9fc95420b8e..fb8b12708b9 100644 --- a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestBindExceptionHandling.java +++ b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestBindExceptionHandling.java @@ -42,7 +42,9 @@ public class TestBindExceptionHandling { createBoundServer(true, false); assertNotNull(thriftServer.tserver); } finally { - thriftServer.stop(); + if (thriftServer != null) { + thriftServer.stop(); + } } } @@ -57,7 +59,9 @@ public class TestBindExceptionHandling { createBoundServer(false, true); assertNotNull(thriftServer.tserver); } finally { - thriftServer.stop(); + if (thriftServer != null) { + thriftServer.stop(); + } } } }