From 6b5511e9919e7020e738132a3c2aaba4207435b6 Mon Sep 17 00:00:00 2001
From: Michael Stack <stack@apache.org>
Date: Thu, 11 Nov 2010 00:38:32 +0000
Subject: [PATCH] HBASE-3222 Regionserver region listing in UI is no longer
 ordered

git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1033787 13f79535-47bb-0310-9956-ffa450edef68
---
 CHANGES.txt                                                 | 1 +
 .../resources/hbase-webapps/regionserver/regionserver.jsp   | 6 ++++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/CHANGES.txt b/CHANGES.txt
index c439bb2be9a..a32e330988e 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -675,6 +675,7 @@ Release 0.90.0 - Unreleased
                failing (Gary via jgray)
    HBASE-3216  Move HBaseFsck from client to util
    HBASE-3219  Split parents are reassigned on restart and on disable/enable
+   HBASE-3222  Regionserver region listing in UI is no longer ordered
 
 
   IMPROVEMENTS
diff --git a/src/main/resources/hbase-webapps/regionserver/regionserver.jsp b/src/main/resources/hbase-webapps/regionserver/regionserver.jsp
index e50188adc95..68d4e422bc0 100644
--- a/src/main/resources/hbase-webapps/regionserver/regionserver.jsp
+++ b/src/main/resources/hbase-webapps/regionserver/regionserver.jsp
@@ -18,7 +18,7 @@
     e.printStackTrace();
   }
   RegionServerMetrics metrics = regionServer.getMetrics();
-  Collection<HRegionInfo> onlineRegions = regionServer.getOnlineRegions();
+  List<HRegionInfo> onlineRegions = regionServer.getOnlineRegions();
   int interval = regionServer.getConfiguration().getInt("hbase.regionserver.msginterval", 3000)/1000;
 
 %><?xml version="1.0" encoding="UTF-8" ?>
@@ -49,7 +49,9 @@
 <% if (onlineRegions != null && onlineRegions.size() > 0) { %>
 <table>
 <tr><th>Region Name</th><th>Start Key</th><th>End Key</th><th>Metrics</th></tr>
-<%   for (HRegionInfo r: onlineRegions) { 
+<%   
+  Collections.sort(onlineRegions);
+  for (HRegionInfo r: onlineRegions) { 
         HServerLoad.RegionLoad load = regionServer.createRegionLoad(r.getEncodedName());
  %>
 <tr><td><%= r.getRegionNameAsString() %></td>