[jira] [Created] (HBASE-23306) Add sanity check for Admin#deleteTables and other operations

["Rahul Kesharwani (Jira)" <ji...@apache.org>]

Rahul Kesharwani created HBASE-23306:
----------------------------------------

             Summary: Add sanity check for Admin#deleteTables and other operations
                 Key: HBASE-23306
                 URL: https://issues.apache.org/jira/browse/HBASE-23306
             Project: HBase
          Issue Type: Bug
          Components: Admin
            Reporter: Rahul Kesharwani


After taking a brief observation, It seems to be the following operation behaves in the opposite manner of what they should be doing in case `null argument. For example, `Admin#listTables(Pattern)` If its argument is `null` then all the tables would be returned. Also this very method is being used by rest of the listed operation like `deleteTables()`, `enableTables()`, `disableTables()` etc.
 * Admin.listTables(Pattern)
 * Admin.listTableNames(Pattern)
 * Admin.deleteTables(Pattern)
 * Admin.enableTables(Pattern)
 * Admin.disableTables(Pattern)
 * Admin.listSnapshots(Pattern)
 * Admin.deleteSnapshots(Pattern)

This could be a very rare scenario but I think it should either throw an exception or behave opposite of what current behavior is.

*If this was not intentional* then I can quickly put together a patch with precondition check for these operations.


 Note: I tried to look for answers but I was not able to find any relevant issue/comments hence logging this issue. Feel free to close if this is intentional behavior.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)