You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2015/04/25 01:37:26 UTC

hbase git commit: HBASE-13555 StackServlet produces 500 error

Repository: hbase
Updated Branches:
  refs/heads/branch-1 16f8e7f55 -> 85d090d81


HBASE-13555 StackServlet produces 500 error


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/85d090d8
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/85d090d8
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/85d090d8

Branch: refs/heads/branch-1
Commit: 85d090d8161080a666b1330a6dc4a473f3b1c87a
Parents: 16f8e7f
Author: tedyu <yu...@gmail.com>
Authored: Fri Apr 24 16:37:25 2015 -0700
Committer: tedyu <yu...@gmail.com>
Committed: Fri Apr 24 16:37:25 2015 -0700

----------------------------------------------------------------------
 .../apache/hadoop/hbase/http/HttpServer.java    | 11 +++++-----
 .../hbase/regionserver/RSDumpServlet.java       | 22 ++++++++++++--------
 2 files changed, 18 insertions(+), 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/85d090d8/hbase-server/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java
index c0bf6f9..1ffd515 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java
@@ -21,7 +21,6 @@ import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InterruptedIOException;
 import java.io.PrintStream;
-import java.io.PrintWriter;
 import java.net.BindException;
 import java.net.InetSocketAddress;
 import java.net.URI;
@@ -1213,11 +1212,11 @@ public class HttpServer implements FilterContainer {
         return;
       }
       response.setContentType("text/plain; charset=UTF-8");
-      PrintWriter out = response.getWriter();
-      PrintStream ps = new PrintStream(response.getOutputStream(), false, "UTF-8");
-      Threads.printThreadInfo(ps, "");
-      ps.flush();
-      out.close();
+      try (PrintStream out = new PrintStream(
+        response.getOutputStream(), false, "UTF-8")) {
+        Threads.printThreadInfo(out, "");
+        out.flush();
+      }
       ReflectionUtils.logThreadInfo(LOG, "jsp requested", 1);
     }
   }

http://git-wip-us.apache.org/repos/asf/hbase/blob/85d090d8/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSDumpServlet.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSDumpServlet.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSDumpServlet.java
index a38ad0e..77b68ec 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSDumpServlet.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSDumpServlet.java
@@ -102,14 +102,18 @@ public class RSDumpServlet extends StateDumpServlet {
 
   public static void dumpQueue(HRegionServer hrs, PrintWriter out)
       throws IOException {
-    // 1. Print out Compaction/Split Queue
-    out.println("Compaction/Split Queue summary: "
-        + hrs.compactSplitThread.toString() );
-    out.println(hrs.compactSplitThread.dumpQueue());
-
-    // 2. Print out flush Queue
-    out.println("\nFlush Queue summary: "
-        + hrs.cacheFlusher.toString());
-    out.println(hrs.cacheFlusher.dumpQueue());
+    if (hrs.compactSplitThread != null) {
+      // 1. Print out Compaction/Split Queue
+      out.println("Compaction/Split Queue summary: "
+          + hrs.compactSplitThread.toString() );
+      out.println(hrs.compactSplitThread.dumpQueue());
+    }
+
+    if (hrs.cacheFlusher != null) {
+      // 2. Print out flush Queue
+      out.println("\nFlush Queue summary: "
+          + hrs.cacheFlusher.toString());
+      out.println(hrs.cacheFlusher.dumpQueue());
+    }
   }
 }