HBASE-23138 Drop_all table by regex fail (#704)

Signed-off-by: Wellington Chevreuil <[email protected]>
(cherry picked from commit d237106)

hbase-shell/src/main/ruby/hbase/admin.rb

@@ -372,7 +372,15 @@ def drop(table_name)
     # Drops a table
     def drop_all(regex)
       pattern = Pattern.compile(regex.to_s)
-      failed = @admin.deleteTables(pattern).map { |t| t.getTableName.getNameAsString }
+      failed = java.util.ArrayList.new
+      @admin.listTableNames(pattern).each do |table_name|
+        begin
+          @admin.deleteTable(table_name)
+        rescue java.io.IOException => e
+          puts puts "table:#{table_name}, error:#{e.toString}"
+          failed.add(table_name)
+        end
+      end
       failed
     end
 

hbase-shell/src/test/ruby/hbase/admin_test.rb

@@ -415,6 +415,10 @@ def teardown
       admin.enable_all(@regex)
       assert(command(:is_enabled, @t1))
       assert(command(:is_enabled, @t2))
+      admin.disable_all(@regex)
+      admin.drop_all(@regex)
+      assert(!command(:exists, @t1))
+      assert(!command(:exists, @t2))
     end
 
     #-------------------------------------------------------------------------------