svn commit: r673000 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/master/ModifyColumn.java

[st...@apache.org]

Author: stack
Date: Mon Jun 30 22:13:45 2008
New Revision: 673000

URL: http://svn.apache.org/viewvc?rev=673000&view=rev
Log:
HBASE-717 alter table broke with new shell returns InvalidColumnNameException

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/master/ModifyColumn.java

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=673000&r1=672999&r2=673000&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Mon Jun 30 22:13:45 2008
@@ -3,6 +3,7 @@
 
   BUG FIXES
    HBASE-718   hbase shell help info
+   HBASE-717   alter table broke with new shell returns InvalidColumnNameException
 
 Release 0.1.3 - 07/25/2008
 

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/master/ModifyColumn.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/master/ModifyColumn.java?rev=673000&r1=672999&r2=673000&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/master/ModifyColumn.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/master/ModifyColumn.java Mon Jun 30 22:13:45 2008
@@ -44,7 +44,7 @@
   protected void postProcessMeta(MetaRegion m, HRegionInterface server)
   throws IOException {
     for (HRegionInfo i: unservedRegions) {
-      if (!i.getTableDesc().hasFamily(columnName)) {
+      if (i.getTableDesc().hasFamily(columnName)) {
         i.getTableDesc().addFamily(descriptor);
         updateRegionInfo(server, m.getRegionName(), i);
       } else { // otherwise, we have an error.
@@ -54,4 +54,4 @@
       }
     }
   }
-}
\ No newline at end of file
+}