HBASE-29596 Migrate Canary Status Jamon page back to JSP

hbase-checkstyle/src/main/resources/hbase/checkstyle-suppressions.xml

@@ -32,7 +32,6 @@
 -->
 <suppressions>
   <suppress checks="." files=".*/generated/.*\.java"/>
-  <suppress checks="." files=".*/generated-jamon/.*\.java"/>
   <suppress checks="MagicNumberCheck" files=".*/src/test/.*\.java"/>
   <suppress checks="VisibilityModifier" files=".*/src/test/.*\.java"/>
   <suppress checks="InterfaceIsTypeCheck" files=".*/src/main/.*\.java"/>

hbase-server/pom.xml

@@ -222,10 +222,6 @@
       <groupId>org.apache.zookeeper</groupId>
       <artifactId>zookeeper</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.jamon</groupId>
-      <artifactId>jamon-runtime</artifactId>
-    </dependency>
     <dependency>
       <groupId>javax.servlet</groupId>
       <artifactId>javax.servlet-api</artifactId>
@@ -479,7 +475,6 @@
             <phase>generate-sources</phase>
             <configuration>
               <sources>
-                <source>${project.build.directory}/generated-jamon</source>
                 <source>${project.build.directory}/generated-sources/java</source>
               </sources>
             </configuration>
@@ -584,27 +579,13 @@
                 <jspcompiler outputdir="${generated.sources}/java" package="org.apache.hadoop.hbase.generated.master" uriroot="${project.build.directory}/jsp/master" webxml="${build.webapps}/master/WEB-INF/web.xml"/>
                 <mkdir dir="${build.webapps}/regionserver/WEB-INF"/>
                 <jspcompiler outputdir="${generated.sources}/java" package="org.apache.hadoop.hbase.generated.regionserver" uriroot="${project.build.directory}/jsp/regionserver" webxml="${build.webapps}/regionserver/WEB-INF/web.xml"/>
+                <mkdir dir="${build.webapps}/canary/WEB-INF"/>
+                <jspcompiler outputdir="${generated.sources}/java" package="org.apache.hadoop.hbase.generated.canary" uriroot="${project.build.directory}/jsp/canary" webxml="${build.webapps}/canary/WEB-INF/web.xml"/>
               </target>
             </configuration>
           </execution>
         </executions>
       </plugin>
-      <plugin>
-        <groupId>org.jamon</groupId>
-        <artifactId>jamon-maven-plugin</artifactId>
-        <executions>
-          <execution>
-            <goals>
-              <goal>translate</goal>
-            </goals>
-            <phase>generate-sources</phase>
-            <configuration>
-              <templateSourceDir>src/main/jamon</templateSourceDir>
-              <templateOutputDir>target/generated-jamon</templateOutputDir>
-            </configuration>
-          </execution>
-        </executions>
-      </plugin>
       <!-- Testing plugins -->
       <plugin>
         <artifactId>maven-surefire-plugin</artifactId>
@@ -937,22 +918,6 @@
               <configuration>
                 <lifecycleMappingMetadata>
                   <pluginExecutions>
-                    <pluginExecution>
-                      <pluginExecutionFilter>
-                        <groupId>org.jamon</groupId>
-                        <artifactId>jamon-maven-plugin</artifactId>
-                        <versionRange>[2.3.4,)</versionRange>
-                        <goals>
-                          <goal>translate</goal>
-                        </goals>
-                      </pluginExecutionFilter>
-                      <action>
-                        <execute>
-                          <runOnIncremental>false</runOnIncremental>
-                          <runOnConfiguration>true</runOnConfiguration>
-                        </execute>
-                      </action>
-                    </pluginExecution>
                     <pluginExecution>
                       <pluginExecutionFilter>
                         <groupId>org.apache.maven.plugins</groupId>
@@ -988,31 +953,6 @@
             </plugin>
           </plugins>
         </pluginManagement>
-        <plugins>
-          <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-eclipse-plugin</artifactId>
-            <configuration>
-              <additionalProjectnatures>
-                <projectnature>org.jamon.project.jamonnature</projectnature>
-              </additionalProjectnatures>
-              <buildcommands>
-                <buildcommand>org.jamon.project.templateBuilder</buildcommand>
-                <buildcommand>org.eclipse.jdt.core.javabuilder</buildcommand>
-                <buildcommand>org.jamon.project.markerUpdater</buildcommand>
-              </buildcommands>
-              <additionalConfig>
-                <file>
-                  <name>.settings/org.jamon.prefs</name>
-                  <content># now
-                    eclipse.preferences.version=1
-                    templateSourceDir=src/main/jamon
-                    templateOutputDir=target/generated-jamon</content>
-                </file>
-              </additionalConfig>
-            </configuration>
-          </plugin>
-        </plugins>
       </build>
     </profile>
   </profiles>

hbase-server/src/main/java/org/apache/hadoop/hbase/tool/CanaryStatusServlet.java

@@ -18,29 +18,19 @@
 package org.apache.hadoop.hbase.tool;
 
 import java.io.IOException;
-import javax.servlet.ServletException;
 import javax.servlet.http.HttpServlet;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
-import org.apache.hadoop.hbase.tmpl.tool.CanaryStatusTmpl;
 import org.apache.yetus.audience.InterfaceAudience;
 
+/**
+ * Only kept for redirecting to canary.jsp.
+ */
 @InterfaceAudience.Private
 public class CanaryStatusServlet extends HttpServlet {
-  @Override
-  protected void doGet(HttpServletRequest req, HttpServletResponse resp)
-    throws ServletException, IOException {
-    CanaryTool.RegionStdOutSink sink =
-      (CanaryTool.RegionStdOutSink) getServletContext().getAttribute("sink");
-    if (sink == null) {
-      throw new ServletException(
-        "RegionStdOutSink is null! The CanaryTool's InfoServer is not initialized correctly");
-    }
-
-    resp.setContentType("text/html");
 
-    CanaryStatusTmpl tmpl = new CanaryStatusTmpl();
-    tmpl.render(resp.getWriter(), sink);
+  @Override
+  protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws IOException {
+    resp.sendRedirect(req.getContextPath() + "/canary.jsp");
   }
-
 }

hbase-server/src/main/java/org/apache/hadoop/hbase/util/CanaryStatusUtil.java

@@ -0,0 +1,42 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.hadoop.hbase.util;
+
+import org.apache.hadoop.hbase.ServerName;
+import org.apache.yetus.audience.InterfaceAudience;
+
+/**
+ * Utility used by the web UI JSP pages.
+ */
+@InterfaceAudience.Private
+public class CanaryStatusUtil {
+
+  private CanaryStatusUtil() {
+    // Do not instantiate.
+  }
+
+  public static String serverNameLink(ServerName serverName) {
+    int infoPort = serverName.getPort() + 1;
+    String url = "//" + serverName.getHostname() + ":" + infoPort + "/";
+    if (infoPort > 0) {
+      return "<a href=\"" + url + "\">" + serverName.getServerName() + "</a>";
+    } else {
+      return serverName.getServerName();
+    }
+  }
+}