[HBASE-15955] Disable action in CatalogJanitor#setEnabled should wait for active cleanup scan to finish

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.0, 2.0.0
• Fix Version/s: 1.3.0, 1.2.2, 1.1.6, 2.0.0
• Component/s: master
• Hadoop Flags: Reviewed
• Assignee: Stephen Yuan Jiang
• Reporter: Stephen Yuan Jiang
• Created: 03/Jun/16 18:35
• Updated: 04/Jun/16 08:41
• Resolved: 03/Jun/16 22:58

Description

When user calls Admin.enableCatalogJanitor(false) to disable the catalog janitor, it expects that the janitor would stop working once the API returns. It is not true, the janitor could have an active scan going on and clean up unused region. The 'false' state would be enforced during the next background runs.

To avoid confusing, if 'CatalogJanitor.enabled' is true and we want to set to false in CatalogJanitor#setEnabled, the function should wait for the on-going active scan to complete.