Author: stack
Date: Mon Sep 20 22:31:19 2010
New Revision: 999150

URL: http://svn.apache.org/viewvc?rev=999150&view=rev
Log:
HBASE-3017  More log pruning

Modified:
    
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
    
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java
    hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java?rev=999150&r1=999149&r2=999150&view=diff
==============================================================================
--- 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
(original)
+++ 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 
Mon Sep 20 22:31:19 2010
@@ -682,19 +682,19 @@ public class AssignmentManager extends Z
     // Get all available servers
     List<HServerInfo> servers = serverManager.getOnlineServersList();
 
-    LOG.info("Assigning " + allRegions.size() + " across " + servers.size() +
-        " servers");
+    LOG.info("Assigning " + allRegions.size() + " regions across " + 
servers.size() +
+      " servers");
 
     // Generate a cluster startup region placement plan
     Map<HServerInfo,List<HRegionInfo>> bulkPlan =
       LoadBalancer.bulkAssignment(allRegions, servers);
 
     // For each server, create OFFLINE nodes and send OPEN RPCs
-    for(Map.Entry<HServerInfo,List<HRegionInfo>> entry : bulkPlan.entrySet()) {
+    for (Map.Entry<HServerInfo,List<HRegionInfo>> entry : bulkPlan.entrySet()) 
{
       HServerInfo server = entry.getKey();
       List<HRegionInfo> regions = entry.getValue();
       LOG.debug("Assigning " + regions.size() + " regions to " + server);
-      for(HRegionInfo region : regions) {
+      for (HRegionInfo region : regions) {
         LOG.debug("Assigning " + region.getRegionNameAsString() + " to " + 
server);
         String regionName = region.getEncodedName();
         RegionPlan plan = new RegionPlan(region, null,server);

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java?rev=999150&r1=999149&r2=999150&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 
(original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 
Mon Sep 20 22:31:19 2010
@@ -27,11 +27,11 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
-import java.text.DecimalFormat;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.util.StringUtils;
 import org.apache.hadoop.hbase.Chore;
 import org.apache.hadoop.hbase.HMsg;
 import org.apache.hadoop.hbase.HRegionInfo;
@@ -94,8 +94,6 @@ public class ServerManager {
 
   private final DeadServer deadservers = new DeadServer();
 
-  private static final DecimalFormat DF = new DecimalFormat("#.##");
-
   /**
    * Dumps into log current stats on dead servers and number of servers
    * TODO: Make this a metric; dump metrics into log.
@@ -112,7 +110,7 @@ public class ServerManager {
       double averageLoad = getAverageLoad();
       String deadServersList = deadservers.toString();
       LOG.info("regionservers=" + numServers +
-        ", averageload=" + DF.format(averageLoad) +
+        ", averageload=" + StringUtils.limitDecimalTo2(averageLoad) +
         ((numDeadServers > 0)?  ("deadservers=" + deadServersList): ""));
     }
   }

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java?rev=999150&r1=999149&r2=999150&view=diff
==============================================================================
--- 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java
 (original)
+++ 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/SplitTransaction.java
 Mon Sep 20 22:31:19 2010
@@ -196,6 +196,9 @@ class SplitTransaction {
       
services.removeFromOnlineRegions(this.parent.getRegionInfo().getEncodedName());
     }
     this.journal.add(JournalEntry.OFFLINED_PARENT);
+    
+    // TODO: If the below were multithreaded would we complete steps in less
+    // elapsed time?  St.Ack 20100920
 
     splitStoreFiles(this.splitdir, hstoreFilesToSplit);
     // splitStoreFiles creates daughter region dirs under the parent splits dir

Modified: hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp?rev=999150&r1=999149&r2=999150&view=diff
==============================================================================
--- hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp (original)
+++ hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp Mon Sep 20 
22:31:19 2010
@@ -1,6 +1,7 @@
 <%@ page contentType="text/html;charset=UTF-8"
   import="java.util.*"
   import="org.apache.hadoop.conf.Configuration"
+  import="org.apache.hadoop.util.StringUtils"
   import="org.apache.hadoop.hbase.util.Bytes"
   import="org.apache.hadoop.hbase.util.JvmVersion"
   import="org.apache.hadoop.hbase.util.FSUtils"
@@ -66,7 +67,7 @@
 <tr><td>Hadoop Version</td><td><%= 
org.apache.hadoop.util.VersionInfo.getVersion() %>, r<%= 
org.apache.hadoop.util.VersionInfo.getRevision() %></td><td>Hadoop version and 
svn revision</td></tr>
 <tr><td>Hadoop Compiled</td><td><%= 
org.apache.hadoop.util.VersionInfo.getDate() %>, <%= 
org.apache.hadoop.util.VersionInfo.getUser() %></td><td>When Hadoop version was 
compiled and by whom</td></tr>
 <tr><td>HBase Root Directory</td><td><%= 
FSUtils.getRootDir(master.getConfiguration()).toString() %></td><td>Location of 
HBase home directory</td></tr>
-<tr><td>Load average</td><td><%= master.getServerManager().getAverageLoad() 
%></td><td>Average number of regions per regionserver. Naive 
computation.</td></tr>
+<tr><td>Load average</td><td><%= 
StringUtils.limitDecimalTo2(master.getServerManager().getAverageLoad()) 
%></td><td>Average number of regions per regionserver. Naive 
computation.</td></tr>
 <%  if (showFragmentation) { %>
         <tr><td>Fragmentation</td><td><%= frags.get("-TOTAL-") != null ? 
frags.get("-TOTAL-").intValue() + "%" : "n/a" %></td><td>Overall fragmentation 
of all tables, including .META. and -ROOT-.</td></tr>
 <%  } %>


Reply via email to