improve code quality by addressing TODO comments

Author: Akanksha-kedia

gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java

@@ -78,8 +78,8 @@ public ClusterStats monitor(ProxyBackendConfiguration backend)
             properties.setProperty("SSL", String.valueOf(parsedUrl.getProtocol().equals("https")));
         }
         catch (MalformedURLException e) {
-            log.error("could not parse backend url %s ", url);
-            return clusterStats.build(); // TODO Invalid configuration should fail
+            log.error("Invalid backend URL configuration: %s", url);
+            throw new IllegalArgumentException("Invalid backend URL: " + url, e);
         }
 
         try (Connection conn = DriverManager.getConnection(jdbcUrl, properties);

gateway-ha/src/test/java/io/trino/gateway/ha/security/TestOIDC.java

@@ -17,6 +17,7 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.google.common.collect.ImmutableMap;
+import io.airlift.log.Logger;
 import io.trino.gateway.ha.HaGatewayLauncher;
 import io.trino.gateway.ha.HaGatewayTestUtils;
 import okhttp3.Cookie;
@@ -62,6 +63,8 @@
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 final class TestOIDC
 {
+    private static final Logger log = Logger.get(TestOIDC.class);
+
     private static final int TTL_ACCESS_TOKEN_IN_SECONDS = 5;
     private static final int TTL_REFRESH_TOKEN_IN_SECONDS = 15;
 
@@ -158,7 +161,7 @@ void setup()
         File testConfigFile =
                 HaGatewayTestUtils.buildGatewayConfig("auth/oauth-test-config.yml", additionalVars);
         String[] args = {testConfigFile.getAbsolutePath()};
-        System.out.println(ROUTER_PORT);
+        log.info("Starting gateway on port: %d", ROUTER_PORT);
         HaGatewayLauncher.main(args);
     }