[HBASE-22758] Remove the unneccesary info cf deletion in DeleteTableProcedure#deleteFromMeta

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.3.0, 2.0.6, 2.2.1, 2.1.6
• Hadoop Flags: Reviewed
• Assignee: Zheng Hu
• Reporter: Zheng Hu
• Created: 28/Jul/19 05:48
• Updated: 31/Jul/19 02:13
• Resolved: 30/Jul/19 14:03

Description

I saw the deleteFromMeta() method in DeleteTableProcedure & TruncateTableProcedure:

protected static void deleteFromMeta(final MasterProcedureEnv env,
    final TableName tableName, List<RegionInfo> regions) throws IOException {
  MetaTableAccessor.deleteRegions(env.getMasterServices().getConnection(), regions);

  // Clean any remaining rows for this table.
  cleanAnyRemainingRows(env, tableName);

  // clean region references from the server manager
  env.getMasterServices().getServerManager().removeRegions(regions);

  // Clear Favored Nodes for this table
  FavoredNodesManager fnm = env.getMasterServices().getFavoredNodesManager();
  if (fnm != null) {
    fnm.deleteFavoredNodesForRegions(regions);
  }
}

1. deleteRegions : delete the info column family first;
2. cleanAnyRemainingRows : delete the whole row for regions then.

In fact, we can only change to delete the whole row, no need the extra info cf deletion any more.

Issue Links

• links to: GitHub Pull Request #424