fix: connection_timeout should not be 0.

Author: youngsofun

databend-client/src/main/java/com/databend/client/ClientSettings.java

@@ -20,7 +20,7 @@
 public class ClientSettings {
     public static final Integer DEFAULT_QUERY_TIMEOUT = 300;
     // seconds
-    public static final Integer DEFAULT_CONNECTION_TIMEOUT = 0;
+    public static final Integer DEFAULT_CONNECTION_TIMEOUT = 60;
     public static final Integer DEFAULT_SOCKET_TIMEOUT = 0;
     public static final int DEFAULT_RETRY_ATTEMPTS = 5;
     public static final String X_Databend_Query_ID = "X-DATABEND-QUERY-ID";

databend-jdbc/src/main/java/com/databend/jdbc/DatabendDriverUri.java

@@ -451,7 +451,9 @@ public void setupClient(OkHttpClient.Builder builder) throws SQLException {
                 builder.addInterceptor(tokenAuth(ACCESS_TOKEN.getValue(properties).get()));
             }
             if (CONNECTION_TIMEOUT.getValue(properties).isPresent()) {
-                builder.connectTimeout(CONNECTION_TIMEOUT.getValue(properties).get(), TimeUnit.SECONDS);
+                Integer timeout =  CONNECTION_TIMEOUT.getValue(properties).get();
+                if (timeout != 0)
+                    builder.connectTimeout(timeout, TimeUnit.SECONDS);
             }
             if (SOCKET_TIMEOUT.getValue(properties).isPresent()) {
                 builder.readTimeout(SOCKET_TIMEOUT.getValue(properties).get(), TimeUnit.SECONDS);