[HBASE-21242] [amv2] Miscellaneous minor log and assign procedure create improvements

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.1.1, 2.0.3
• Component/s: amv2, Operability
• Assignee: Michael Stack
• Reporter: Michael Stack
• Created: 27/Sep/18 04:31
• Updated: 20/Oct/18 05:13
• Resolved: 20/Oct/18 05:13

Description

Some minor fixups:

    For RIT Duration, do better than print ms/seconds. Remove redundant UI
    column dedicated to duration when we log it in the status field too.

    Make bypass log at INFO level -- when DEBUG we can miss important
    fixup detail like why we failed.

    Make it so on complete of subprocedure, we note count of outstanding
    siblings so we have a clue how much further the parent has to go before
    it is done (Helpful when hundreds of servers doing SCP).

    Have the SCP run the AP preflight check before creating an AP; saves
    creation of hundreds of thousands of APs during fixup of this big cluster
    of mine.

    Don't log tablename three times when reporting remote call failed.

    If lock is held already, note who has it. Also log after we get lock
    or if we have to wait rather than log on entrance though we may
    later have to wait (or we may have just picked up the lock).

Posting patch in a sec but let me try it on cluster too.

Issue Links

• breaks: HBASE-21305 TestHRegionInfo failing because descriptive strings have different times
• links to: Review Board (branch-2)
• links to: Review Board (branch-2.0)
• links to: Review Board (branch-2.1)