PDavid commented on code in PR #6875:
URL: https://github.com/apache/hbase/pull/6875#discussion_r2332673448


##########
hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/RSStatusTmpl.jamon:
##########
@@ -243,7 +243,7 @@ org.apache.hadoop.hbase.zookeeper.MasterAddressTracker;
                 <%else>
                 <%java>
                 String host = masterServerName.getHostname() + ":" + infoPort;
-                String url = "//" + host + "/master-status";
+                String url = "//" + host + "/master.jsp";

Review Comment:
   This PR is already quite huge and it is just the master status page. I 
wanted to do the migration step-by step so migrating the RegionServer status 
page will be the next under https://issues.apache.org/jira/browse/HBASE-29531 
and under a different PR.
   
   Although the biggest Jamon code was the Master Status page so the rest would 
be smaller.
   
   What do you think, should we rather do all the Jamon migration under one 
ticket and PR?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to