svn commit: r1181470 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

[ns...@apache.org]

Author: nspiegelberg
Date: Tue Oct 11 02:11:31 2011
New Revision: 1181470

URL: http://svn.apache.org/viewvc?rev=1181470&view=rev
Log:
Throw Exception on the client side if the region is improperly formatted.

Modified:
    hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java?rev=1181470&r1=1181469&r2=1181470&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java Tue Oct 11 02:11:31 2011
@@ -805,11 +805,12 @@ public class HBaseAdmin {
     if (tableNameOrRegionName == null) {
       throw new IllegalArgumentException("Pass a table name or region name");
     }
-    byte [] tableName = tableExists(tableNameOrRegionName)?
-      tableNameOrRegionName: null;
-    byte [] regionName = tableName == null? tableNameOrRegionName: null;
-    Object [] args = regionName == null? null: new byte [][] {regionName};
-    modifyTable(tableName == null? null: tableName, op, args);
+    if (tableExists(tableNameOrRegionName)) {
+      modifyTable(tableNameOrRegionName, op, null);
+    } else {
+      HRegionInfo.parseRegionName(tableNameOrRegionName); // verify format
+      modifyTable(null, op, new byte[][] {tableNameOrRegionName});
+    }
   }
 
   /**