[HBASE-14998] Unify synchronous and asynchronous methods in Admin and cleanup

• Type: Sub-task
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 2.0.0-alpha-3, 2.0.0
• Hadoop Flags: Reviewed

Release Note

 * Deprecates getAlterStatus. Everywhere else we talk of 'modify' rather
       'alter' and should use Future returned from async instead.
 * isTableAvailable(TableName, byte [][]) has been deprecated to be
       removed; use the overrie instead. This is a weird method.
 * Changed listTableDescriptor to getDescriptor.
 * Renamed other like methods to have same pattern (deprecating the old):
        balancer => balance
        setBalancerRunning => balancerSwitch
        setNormalizerRunning => normalizerSwitch
        enableCatalogJanitor => catalogJanitorSwitch
        setCleanerChoreRunning => cleanerChoreSwitch
        setSplitOrMergeEnabled => splitOrMergeEnabledSwitch

 * Renamed (with deprecation of old) runCatalogScan => runCatalogJanitor.
 * Reviewed generated javadoc and made some edits; purged reference to
       hbase issues from our API, fixed param names, etc.
 * Made all the enable services methods have same pattern.
 * Renamed takeSnapshotAsync as snapshotAsync (with deprecation of old)
 * Renamed execProcedureWithRet as execProcedureWithReturn (with
       deprecation)

• Assignee: Balazs Meszaros
• Reporter: Enis Soztutar
• Created: 16/Dec/15 22:38
• Updated: 04/Oct/19 14:59
• Resolved: 14/Sep/17 00:22

Description

Admin has a bunch of methods, some are asnyc, some are sync. Needs some unification in method naming, and method signatures.

• We use modify and alter interchangeably. Pick one and stick with it (modifyTable(), versus getAlterStatus()). Shell uses alter.
• Remove getAlterStatus(), should not be needed.
• remove already deprecated methods
• isTableAvailable(TableName tableName, byte[][] splitKeys) should be removed.
• Consistently use Aysnc as a prefix for all async methods.
• Other ideas?