Add checks to prevent loading default modules in config

Author: oneonestar

gateway-ha/src/main/java/io/trino/gateway/baseapp/BaseApp.java

@@ -13,6 +13,7 @@
  */
 package io.trino.gateway.baseapp;
 
+import com.google.common.collect.ImmutableList;
 import com.google.inject.Binder;
 import com.google.inject.Module;
 import com.google.inject.Scopes;
@@ -107,6 +108,7 @@ public static List<Module> addModules(HaGatewayConfiguration configuration)
         }
         else {
             for (String clazz : configuration.getModules()) {
+                warnLoadingDefaultModules(clazz);
                 modules.add(newModule(clazz, configuration));
             }
         }
@@ -115,6 +117,19 @@ public static List<Module> addModules(HaGatewayConfiguration configuration)
         return modules;
     }
 
+    private static void warnLoadingDefaultModules(String clazz)
+    {
+        // Remove this check when user finished the migration
+        List<String> defaultModules = ImmutableList.of(
+                "io.trino.gateway.ha.module.HaGatewayProviderModule",
+                "io.trino.gateway.ha.module.ClusterStateListenerModule",
+                "io.trino.gateway.ha.module.ClusterStatsMonitorModule");
+        if (defaultModules.contains(clazz)) {
+            logger.error("Default module [%s] is already being loaded. Please remove it from the config file", clazz);
+            System.exit(1);
+        }
+    }
+
     @Override
     public void configure(Binder binder)
     {
@@ -138,6 +153,11 @@ private static void addManagedApps(HaGatewayConfiguration configuration, Binder
         }
         configuration.getManagedApps().forEach(
                 clazz -> {
+                    // Remove this check when user finished the migration
+                    if (clazz.equals("io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor")) {
+                        logger.error("Default class ActiveClusterMonitor is already being loaded. Please remove it from the config file");
+                        System.exit(1);
+                    }
                     try {
                         Class<?> c = Class.forName(clazz);
                         binder.bind(c).in(Scopes.SINGLETON);