Return user visible error when catalog is misconfigured

When Driver::connect returns null, most likely it catalog is
misconfigured and error message from the checkState was propagated to
the user making looking obscure.

Before:
Query 20250825_145634_00011_9263i failed: Error listing schemas for
catalog postgresql2:
com.google.common.util.concurrent.UncheckedExecutionException:
java.lang.IllegalStateException: Driver returned null connection, make
sure the connection URL 'jdbc:psql://test' is valid for the driver
org.postgresql.Driver@63a777ba

After:
Query 20250825_150832_00003_52kd7 failed: Error listing schemas for
catalog postgresql: Driver returned null connection, make sure the
connection URL 'jdbc:jdbc:postgresql://localhost:5432/tpch' and
credentials are valid

Author: kokosing

plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/TracingDataSource.java

@@ -15,6 +15,7 @@
 
 import io.opentelemetry.api.OpenTelemetry;
 import io.opentelemetry.instrumentation.jdbc.datasource.JdbcTelemetry;
+import io.trino.spi.TrinoException;
 
 import javax.sql.DataSource;
 
@@ -25,7 +26,7 @@
 import java.util.Properties;
 import java.util.logging.Logger;
 
-import static com.google.common.base.Preconditions.checkState;
+import static io.trino.plugin.jdbc.JdbcErrorCode.JDBC_ERROR;
 import static java.util.Objects.requireNonNull;
 
 public class TracingDataSource
@@ -67,7 +68,9 @@ public Connection getConnection()
                 throws SQLException
         {
             Connection connection = driver.connect(connectionUrl, properties);
-            checkState(connection != null, "Driver returned null connection, make sure the connection URL '%s' is valid for the driver %s", connectionUrl, driver);
+            if (connection == null) {
+                throw new TrinoException(JDBC_ERROR, "Driver returned null connection, make sure the connection URL '%s' and credentials are valid".formatted(connectionUrl));
+            }
             return connection;
         }