[HBASE-20627] Relocate RS Group pre/post hooks from RSGroupAdminServer to RSGroupAdminEndpoint

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.1.0, 1.4.5
• Hadoop Flags: Reviewed
• Assignee: Ted Yu
• Reporter: Ted Yu
• Created: 23/May/18 16:18
• Updated: 30/Sep/18 00:00
• Resolved: 24/May/18 01:43

Description

Currently RS Group pre/post hooks are called from RSGroupAdminServer.
e.g. RSGroupAdminServer#removeRSGroup :

      if (master.getMasterCoprocessorHost() != null) {
        master.getMasterCoprocessorHost().preRemoveRSGroup(name);
      }

RSGroupAdminServer#removeRSGroup is called by RSGroupAdminEndpoint :

        checkPermission("removeRSGroup");
        groupAdminServer.removeRSGroup(request.getRSGroupName());

If permission check fails, the pre hook wouldn't be called.