Throw exception when failed to parse backend URL config

Akanksha-kedia · ebyhr · commit 34d27ef2f3ef · 2025-10-02T06:12:30.000+09:00
diff --git a/gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java b/gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java
@@ -78,8 +78,7 @@ 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
+            throw new IllegalArgumentException("Invalid backend URL: " + url, e);
         }
 
         try (Connection conn = DriverManager.getConnection(jdbcUrl, properties);

Original file line number	Diff line number	Diff line change
`@@ -78,8 +78,7 @@ public ClusterStats monitor(ProxyBackendConfiguration backend)`
`78`	`78`	`properties.setProperty("SSL", String.valueOf(parsedUrl.getProtocol().equals("https")));`
`79`	`79`	`}`
`80`	`80`	`catch (MalformedURLException e) {`
`81`		`- log.error("could not parse backend url %s ", url);`
`82`		`- return clusterStats.build(); // TODO Invalid configuration should fail`
	`81`	`+ throw new IllegalArgumentException("Invalid backend URL: " + url, e);`
`83`	`82`	`}`
`84`	`83`
`85`	`84`	`try (Connection conn = DriverManager.getConnection(jdbcUrl, properties);`