You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2008/06/12 07:56:42 UTC

svn commit: r666962 - /hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

Author: stack
Date: Wed Jun 11 22:56:41 2008
New Revision: 666962

URL: http://svn.apache.org/viewvc?rev=666962&view=rev
Log:
toString tablename when logging or throwing exception

Modified:
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java?rev=666962&r1=666961&r2=666962&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/client/HBaseAdmin.java Wed Jun 11 22:56:41 2008
@@ -280,7 +280,7 @@
         // continue
       }
     }
-    LOG.info("table " + tableName + " deleted");
+    LOG.info("Deleted " + Bytes.toString(tableName));
   }
 
   /**
@@ -344,7 +344,7 @@
           if (values == null || values.size() == 0) {
             if (valuesfound == 0) {
               throw new NoSuchElementException(
-                  "table " + tableName + " not found");
+                  "table " + Bytes.toString(tableName) + " not found");
             }
             break;
           }
@@ -386,7 +386,7 @@
       }
       if (LOG.isDebugEnabled()) {
         LOG.debug("Sleep. Waiting for first region to be enabled from " +
-            tableName);
+          Bytes.toString(tableName));
       }
       try {
         Thread.sleep(pause);
@@ -396,10 +396,10 @@
       }
       if (LOG.isDebugEnabled()) {
         LOG.debug("Wake. Waiting for first region to be enabled from " +
-            tableName);
+          Bytes.toString(tableName));
       }
     }
-    LOG.info("Enabled table " + tableName);
+    LOG.info("Enabled table " + Bytes.toString(tableName));
   }
 
   /**
@@ -462,7 +462,8 @@
           RowResult values = server.next(scannerId);
           if (values == null || values.size() == 0) {
             if (valuesfound == 0) {
-              throw new NoSuchElementException("table " + tableName + " not found");
+              throw new NoSuchElementException("table " +
+                Bytes.toString(tableName) + " not found");
             }
             break;
           }
@@ -504,7 +505,7 @@
       }
       if (LOG.isDebugEnabled()) {
         LOG.debug("Sleep. Waiting for first region to be disabled from " +
-            tableName);
+          Bytes.toString(tableName));
       }
       try {
         Thread.sleep(pause);
@@ -516,7 +517,7 @@
             tableName);
       }
     }
-    LOG.info("Disabled table " + tableName);
+    LOG.info("Disabled " + Bytes.toString(tableName));
   }
   
   /**