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/07 07:27:13 UTC

svn commit: r664284 - in /hadoop/hbase/branches/0.1: CHANGES.txt src/java/org/apache/hadoop/hbase/HMaster.java

Author: stack
Date: Fri Jun  6 22:27:13 2008
New Revision: 664284

URL: http://svn.apache.org/viewvc?rev=664284&view=rev
Log:
HBASE-652 dropping table fails silently if table isn't disabled

Modified:
    hadoop/hbase/branches/0.1/CHANGES.txt
    hadoop/hbase/branches/0.1/src/java/org/apache/hadoop/hbase/HMaster.java

Modified: hadoop/hbase/branches/0.1/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.1/CHANGES.txt?rev=664284&r1=664283&r2=664284&view=diff
==============================================================================
--- hadoop/hbase/branches/0.1/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.1/CHANGES.txt Fri Jun  6 22:27:13 2008
@@ -10,6 +10,7 @@
    HBASE-648   If mapfile index is empty, run repair
    HBASE-659   HLog#cacheFlushLock not cleared; hangs a region
    HBASE-663   Incorrect sequence number for cache flush
+   HBASE-652   Dropping table fails silently if table isn't disabled 
 
 Release 0.1.2 - 05/13/2008
 

Modified: hadoop/hbase/branches/0.1/src/java/org/apache/hadoop/hbase/HMaster.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.1/src/java/org/apache/hadoop/hbase/HMaster.java?rev=664284&r1=664283&r2=664284&view=diff
==============================================================================
--- hadoop/hbase/branches/0.1/src/java/org/apache/hadoop/hbase/HMaster.java (original)
+++ hadoop/hbase/branches/0.1/src/java/org/apache/hadoop/hbase/HMaster.java Fri Jun  6 22:27:13 2008
@@ -2958,7 +2958,8 @@
         final HRegionInfo info) throws IOException {
       
       if (isEnabled(info)) {
-        throw new TableNotDisabledException(tableName.toString());
+        throw new TableNotDisabledException("You must first disable table <" +
+          tableName.toString() + "> before you can drop it");
       }
     }