Revert "HBASE-22729 Start RSGroupInfoManager as default (#555)"

This reverts commit 58dee3e.

Author: Apache9

hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java

@@ -186,7 +186,6 @@
 import org.apache.hadoop.hbase.replication.master.ReplicationLogCleaner;
 import org.apache.hadoop.hbase.replication.master.ReplicationPeerConfigUpgrader;
 import org.apache.hadoop.hbase.replication.regionserver.ReplicationStatus;
-import org.apache.hadoop.hbase.rsgroup.RSGroupInfoManager;
 import org.apache.hadoop.hbase.security.AccessDeniedException;
 import org.apache.hadoop.hbase.security.SecurityConstants;
 import org.apache.hadoop.hbase.security.UserProvider;
@@ -351,8 +350,6 @@ public void run() {
   // manager of assignment nodes in zookeeper
   private AssignmentManager assignmentManager;
 
-  private RSGroupInfoManager rsGroupInfoManager;
-
   // manager of replication
   private ReplicationPeerManager replicationPeerManager;
 
@@ -775,8 +772,6 @@ protected void initializeZKBasedSystemTrackers()
     this.splitOrMergeTracker = new SplitOrMergeTracker(zooKeeper, conf, this);
     this.splitOrMergeTracker.start();
 
-    this.rsGroupInfoManager = RSGroupInfoManager.create(this);
-
     this.replicationPeerManager = ReplicationPeerManager.create(zooKeeper, conf);
 
     this.drainingServerTracker = new DrainingServerTracker(zooKeeper, this, this.serverManager);
@@ -3756,9 +3751,4 @@ public Map<String, ReplicationStatus> getWalGroupsReplicationStatus() {
   public HbckChore getHbckChore() {
     return this.hbckChore;
   }
-
-  @Override
-  public RSGroupInfoManager getRSRSGroupInfoManager() {
-    return rsGroupInfoManager;
-  }
 }

hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterServices.java

@@ -18,8 +18,10 @@
 package org.apache.hadoop.hbase.master;
 
 import com.google.protobuf.Service;
+
 import java.io.IOException;
 import java.util.List;
+
 import org.apache.hadoop.hbase.Server;
 import org.apache.hadoop.hbase.ServerName;
 import org.apache.hadoop.hbase.TableDescriptors;
@@ -49,7 +51,6 @@
 import org.apache.hadoop.hbase.replication.ReplicationPeerConfig;
 import org.apache.hadoop.hbase.replication.ReplicationPeerDescription;
 import org.apache.hadoop.hbase.replication.SyncReplicationState;
-import org.apache.hadoop.hbase.rsgroup.RSGroupInfoManager;
 import org.apache.hadoop.hbase.security.access.AccessChecker;
 import org.apache.hadoop.hbase.security.access.ZKPermissionWatcher;
 import org.apache.yetus.audience.InterfaceAudience;
@@ -536,8 +537,4 @@ default SplitWALManager getSplitWALManager(){
    */
   List<RegionPlan> executeRegionPlansWithThrottling(List<RegionPlan> plans);
 
-  /**
-   * @return the {@link RSGroupInfoManager}
-   */
-  RSGroupInfoManager getRSRSGroupInfoManager();
 }

hbase-server/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java

@@ -66,7 +66,7 @@ public void start(CoprocessorEnvironment env) throws IOException {
     }
 
     master = ((HasMasterServices) env).getMasterServices();
-    groupInfoManager = master.getRSRSGroupInfoManager();
+    groupInfoManager = RSGroupInfoManagerImpl.getInstance(master);
     groupAdminServer = new RSGroupAdminServer(master, groupInfoManager);
     Class<?> clazz =
       master.getConfiguration().getClass(HConstants.HBASE_MASTER_LOADBALANCER_CLASS, null);

hbase-server/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupInfoManager.java

@@ -21,7 +21,6 @@
 import java.util.List;
 import java.util.Set;
 import org.apache.hadoop.hbase.TableName;
-import org.apache.hadoop.hbase.master.MasterServices;
 import org.apache.hadoop.hbase.net.Address;
 import org.apache.yetus.audience.InterfaceAudience;
 
@@ -88,8 +87,4 @@ Set<Address> moveServers(Set<Address> servers, String srcGroup, String dstGroup)
    */
   @Deprecated
   RSGroupInfo getRSGroupForTable(TableName tableName) throws IOException;
-
-  static RSGroupInfoManager create(MasterServices master) throws IOException {
-    return RSGroupInfoManagerImpl.getInstance(master);
-  }
 }

hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockNoopMasterServices.java

@@ -54,7 +54,6 @@
 import org.apache.hadoop.hbase.replication.ReplicationPeerConfig;
 import org.apache.hadoop.hbase.replication.ReplicationPeerDescription;
 import org.apache.hadoop.hbase.replication.SyncReplicationState;
-import org.apache.hadoop.hbase.rsgroup.RSGroupInfoManager;
 import org.apache.hadoop.hbase.security.access.AccessChecker;
 import org.apache.hadoop.hbase.security.access.ZKPermissionWatcher;
 import org.apache.hadoop.hbase.zookeeper.ZKWatcher;
@@ -491,9 +490,4 @@ public List<RegionPlan> executeRegionPlansWithThrottling(List<RegionPlan> plans)
   public AsyncClusterConnection getAsyncClusterConnection() {
     return null;
   }
-
-  @Override
-  public RSGroupInfoManager getRSRSGroupInfoManager() {
-    return null;
-  }
 }