[ 
https://issues.apache.org/jira/browse/HBASE-3506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13052840#comment-13052840
 ] 

Ted Yu commented on HBASE-3506:
-------------------------------

Nice work Joey.

I think we should catch exception(s) coming out of deleteTable() so that an 
exception in the middle of the operation doesn't stop other tables from being 
dropped:
{code}
+  public void deleteTables(Pattern pattern) throws IOException {
+    for (HTableDescriptor table : listTables(pattern)) {
+      deleteTable(table.getName());
+    }
+  }
{code}
Also, it would be nice to change the return type to HTableDescriptor[] which 
contain the table(s) that might have not been deleted.

Similar comment applies to disableTables().


> Allow table name expressed in regex in drop, disable, enable operations
> -----------------------------------------------------------------------
>
>                 Key: HBASE-3506
>                 URL: https://issues.apache.org/jira/browse/HBASE-3506
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Ted Yu
>            Assignee: Joey Echeverria
>         Attachments: HBASE-3506-1.patch
>
>
> Ability to disable, drop and enable tables using regex expression is 
> desirable.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to