You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by jd...@apache.org on 2008/08/08 20:53:30 UTC

svn commit: r684053 - in /hadoop/hbase/trunk: CHANGES.txt bin/HBase.rb

Author: jdcryans
Date: Fri Aug  8 11:53:30 2008
New Revision: 684053

URL: http://svn.apache.org/viewvc?rev=684053&view=rev
Log:
Added a check in HBase.rb to make sure table is disabled when drop.

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/bin/HBase.rb

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=684053&r1=684052&r2=684053&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Fri Aug  8 11:53:30 2008
@@ -5,6 +5,9 @@
   INCOMPATIBLE CHANGES
   BUG FIXES
   IMPROVEMENTS
+   HBASE-801  When a table haven't disable, shell could response in a "user
+              friendly" way.
+
   NEW FEATURES
   OPTIMIZATIONS
 

Modified: hadoop/hbase/trunk/bin/HBase.rb
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/bin/HBase.rb?rev=684053&r1=684052&r2=684053&view=diff
==============================================================================
--- hadoop/hbase/trunk/bin/HBase.rb (original)
+++ hadoop/hbase/trunk/bin/HBase.rb Fri Aug  8 11:53:30 2008
@@ -89,8 +89,12 @@
 
     def drop(tableName)
       now = Time.now 
-      @admin.deleteTable(tableName)
       @formatter.header()
+      if @admin.isTableEnabled(tableName)
+        raise IOError.new("Table " + tableName + " is enabled. Disable it first")
+      else
+        @admin.deleteTable(tableName)
+      end
       @formatter.footer(now)
     end