[feature-#1955]jdbcConf adds default optimization parameters (#1959)

yanghuaiGit · dujie · web-flow · commit c956d2508c98 · 2025-12-02T08:24:07.000+08:00
Co-authored-by: dujie &lt;dujie@dtstack.com&gt;
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -71,11 +71,6 @@ jobs:
           distribution: 'temurin'
           cache: 'maven'
       - name: add dependencies
-        run: |
-          wget http://nexus.dev.dtstack.cn/nexus/content/repositories/dtstack-release/com/esen/jdbc/gbase/8.3.81.53/gbase-8.3.81.53.jar
-          wget https://cdn.gbase.cn/products/27/czrl6z38BvTfEQS4uyQcS/gbasedbtjdbc_3.5.1.jar
-          ./mvnw install:install-file -DgroupId=com.esen.jdbc -DartifactId=gbase -Dversion=8.3.81.53 -Dpackaging=jar -Dfile=./gbase-8.3.81.53.jar
-          ./mvnw install:install-file -DgroupId=com.gbasedbt.jdbc.Driver -DartifactId=gbasedbt -Dversion=3.5.1_1_d0c87a -Dpackaging=jar -Dfile=./gbasedbtjdbc_3.5.1.jar
       - name: build project
         run: |
           ./mvnw clean package -Dmaven.test.skip --no-snapshot-updates
diff --git a/chunjun-connectors/chunjun-connector-clickhouse/src/main/java/com/dtstack/chunjun/connector/clickhouse/converter/ClickhouseRawTypeConverter.java b/chunjun-connectors/chunjun-connector-clickhouse/src/main/java/com/dtstack/chunjun/connector/clickhouse/converter/ClickhouseRawTypeConverter.java
@@ -40,6 +40,7 @@ public class ClickhouseRawTypeConverter {
     public static DataType apply(TypeConfig type) {
         switch (type.getType().toUpperCase(Locale.ENGLISH)) {
             case "BOOLEAN":
+            case "BOOL":
                 return DataTypes.BOOLEAN();
             case "TINYINT":
             case "INT8":
diff --git a/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/dialect/JdbcDialect.java b/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/dialect/JdbcDialect.java
@@ -525,4 +525,18 @@ default Function<Tuple3<String, Integer, Integer>, TypeConfig> typeBuilder() {
     default TableIdentify getTableIdentify(String confSchema, String confTable) {
         return new TableIdentify(null, confSchema, confTable, this::quoteIdentifier, false);
     }
+
+    /**
+     * Add additional parameters to jdbc properties,for reader only.
+     *
+     * @param jdbcConf jdbc datasource configuration
+     */
+    default void putReaderExtParam(JdbcConfig jdbcConf) {}
+
+    /**
+     * Add additional parameters to jdbc properties,for writer only.
+     *
+     * @param jdbcConf jdbc datasource configuration
+     */
+    default void putWriterExtParam(JdbcConfig jdbcConf) {}
 }
diff --git a/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/sink/JdbcSinkFactory.java b/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/sink/JdbcSinkFactory.java
@@ -96,6 +96,7 @@ public JdbcSinkFactory(SyncConfig syncConfig, JdbcDialect jdbcDialect) {
 
     @Override
     public DataStreamSink<RowData> createSink(DataStream<RowData> dataSet) {
+        jdbcDialect.putWriterExtParam(jdbcConfig);
         JdbcOutputFormatBuilder builder = getBuilder();
         initColumnInfo();
         builder.setJdbcConf(jdbcConfig);
diff --git a/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/source/JdbcSourceFactory.java b/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/source/JdbcSourceFactory.java
@@ -105,6 +105,7 @@ protected Class<? extends JdbcConfig> getConfClass() {
 
     @Override
     public DataStream<RowData> createSource() {
+        jdbcDialect.putReaderExtParam(jdbcConfig);
         initColumnInfo();
         initRestoreConfig();
         initPollingConfig();
diff --git a/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/table/JdbcDynamicTableFactory.java b/chunjun-connectors/chunjun-connector-jdbc-base/src/main/java/com/dtstack/chunjun/connector/jdbc/table/JdbcDynamicTableFactory.java
@@ -185,6 +185,7 @@ protected JdbcConfig getSinkConnectionConfig(
 
         jdbcConfig.setUniqueKey(keyFields);
         resetTableInfo(jdbcConfig);
+        getDialect().putWriterExtParam(jdbcConfig);
         return jdbcConfig;
     }
 
@@ -257,6 +258,7 @@ protected JdbcConfig getSourceConnectionConfig(ReadableConfig readableConfig) {
         if (StringUtils.isBlank(jdbcConfig.getCustomSql())) {
             resetTableInfo(jdbcConfig);
         }
+        getDialect().putReaderExtParam(jdbcConfig);
         return jdbcConfig;
     }
 
diff --git a/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/dialect/MysqlDialect.java b/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/dialect/MysqlDialect.java
@@ -20,6 +20,7 @@
 
 import com.dtstack.chunjun.config.TypeConfig;
 import com.dtstack.chunjun.connector.jdbc.conf.TableIdentify;
+import com.dtstack.chunjun.connector.jdbc.config.JdbcConfig;
 import com.dtstack.chunjun.connector.jdbc.dialect.JdbcDialect;
 import com.dtstack.chunjun.connector.jdbc.statement.FieldNamedPreparedStatement;
 import com.dtstack.chunjun.connector.mysql.converter.MysqlRawTypeConverter;
@@ -37,6 +38,7 @@
 import java.util.Arrays;
 import java.util.Locale;
 import java.util.Optional;
+import java.util.Properties;
 import java.util.function.Function;
 import java.util.stream.Collectors;
 
@@ -166,4 +168,28 @@ public Function<Tuple3<String, Integer, Integer>, TypeConfig> typeBuilder() {
             return typeConfig;
         });
     }
+
+    @Override
+    public void putWriterExtParam(JdbcConfig jdbcConf) {
+        Properties properties = jdbcConf.getProperties();
+        if (properties == null) {
+            properties = new Properties();
+        }
+        properties.putIfAbsent("useCursorFetch", "true");
+        properties.putIfAbsent("rewriteBatchedStatements", "true");
+        properties.put("tinyInt1isBit", "false");
+        jdbcConf.setProperties(properties);
+    }
+
+    @Override
+    public void putReaderExtParam(JdbcConfig jdbcConf) {
+        Properties properties = jdbcConf.getProperties();
+        if (properties == null) {
+            properties = new Properties();
+        }
+        properties.putIfAbsent("useCursorFetch", "true");
+        properties.putIfAbsent("rewriteBatchedStatements", "true");
+        properties.put("tinyInt1isBit", "false");
+        jdbcConf.setProperties(properties);
+    }
 }
diff --git a/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/sink/MysqlSinkFactory.java b/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/sink/MysqlSinkFactory.java
@@ -22,14 +22,12 @@
 import com.dtstack.chunjun.connector.jdbc.sink.JdbcOutputFormat;
 import com.dtstack.chunjun.connector.jdbc.sink.JdbcOutputFormatBuilder;
 import com.dtstack.chunjun.connector.jdbc.sink.JdbcSinkFactory;
-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
 import com.dtstack.chunjun.connector.mysql.dialect.MysqlDialect;
 
 public class MysqlSinkFactory extends JdbcSinkFactory {
 
     public MysqlSinkFactory(SyncConfig syncConfig) {
         super(syncConfig, new MysqlDialect());
-        JdbcUtil.putExtParam(jdbcConfig);
     }
 
     @Override
diff --git a/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/source/MysqlSourceFactory.java b/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/source/MysqlSourceFactory.java
@@ -22,7 +22,6 @@
 import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormat;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormatBuilder;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcSourceFactory;
-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
 import com.dtstack.chunjun.connector.mysql.dialect.MysqlDialect;
 
 import org.apache.flink.streaming.api.environment.StreamExecutionEnvironment;
@@ -42,7 +41,6 @@ public MysqlSourceFactory(SyncConfig syncConfig, StreamExecutionEnvironment env)
                 && jdbcConfig.getFetchSize() == 0) {
             jdbcConfig.setFetchSize(1000);
         }
-        JdbcUtil.putExtParam(jdbcConfig);
     }
 
     @Override
diff --git a/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/table/MysqlDynamicTableFactory.java b/chunjun-connectors/chunjun-connector-mysql/src/main/java/com/dtstack/chunjun/connector/mysql/table/MysqlDynamicTableFactory.java
@@ -18,19 +18,14 @@
 
 package com.dtstack.chunjun.connector.mysql.table;
 
-import com.dtstack.chunjun.connector.jdbc.config.JdbcConfig;
 import com.dtstack.chunjun.connector.jdbc.dialect.JdbcDialect;
 import com.dtstack.chunjun.connector.jdbc.sink.JdbcOutputFormat;
 import com.dtstack.chunjun.connector.jdbc.sink.JdbcOutputFormatBuilder;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormat;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormatBuilder;
 import com.dtstack.chunjun.connector.jdbc.table.JdbcDynamicTableFactory;
-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
 import com.dtstack.chunjun.connector.mysql.dialect.MysqlDialect;
 
-import org.apache.flink.configuration.ReadableConfig;
-import org.apache.flink.table.catalog.ResolvedSchema;
-
 public class MysqlDynamicTableFactory extends JdbcDynamicTableFactory {
 
     // 默认是Mysql流式拉取
@@ -54,21 +49,6 @@ protected int getDefaultFetchSize() {
         return DEFAULT_FETCH_SIZE;
     }
 
-    @Override
-    protected JdbcConfig getSourceConnectionConfig(ReadableConfig readableConfig) {
-        JdbcConfig jdbcConfig = super.getSourceConnectionConfig(readableConfig);
-        JdbcUtil.putExtParam(jdbcConfig);
-        return jdbcConfig;
-    }
-
-    @Override
-    protected JdbcConfig getSinkConnectionConfig(
-            ReadableConfig readableConfig, ResolvedSchema schema) {
-        JdbcConfig jdbcConfig = super.getSinkConnectionConfig(readableConfig, schema);
-        JdbcUtil.putExtParam(jdbcConfig);
-        return jdbcConfig;
-    }
-
     @Override
     protected JdbcInputFormatBuilder getInputFormatBuilder() {
         return new JdbcInputFormatBuilder(new JdbcInputFormat());
diff --git a/chunjun-connectors/chunjun-connector-mysqld/src/main/java/com/dtstack/chunjun/connector/mysqld/source/MysqldSourceFactory.java b/chunjun-connectors/chunjun-connector-mysqld/src/main/java/com/dtstack/chunjun/connector/mysqld/source/MysqldSourceFactory.java
@@ -24,7 +24,6 @@
 import com.dtstack.chunjun.connector.jdbc.source.distribute.DistributedJdbcInputFormat;
 import com.dtstack.chunjun.connector.jdbc.source.distribute.DistributedJdbcInputFormatBuilder;
 import com.dtstack.chunjun.connector.jdbc.source.distribute.DistributedJdbcSourceFactory;
-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
 import com.dtstack.chunjun.connector.mysql.dialect.MysqlDialect;
 import com.dtstack.chunjun.connector.mysqld.utils.MySqlDataSource;
 import com.dtstack.chunjun.throwable.ChunJunRuntimeException;
@@ -57,7 +56,6 @@ public class MysqldSourceFactory extends DistributedJdbcSourceFactory {
 
     public MysqldSourceFactory(SyncConfig syncConfig, StreamExecutionEnvironment env) {
         super(syncConfig, env, new MysqlDialect());
-        JdbcUtil.putExtParam(jdbcConfig);
     }
 
     @Override
diff --git a/chunjun-connectors/chunjun-connector-oraclelogminer/src/main/java/com/dtstack/chunjun/connector/oraclelogminer/converter/OracleRawTypeMapper.java b/chunjun-connectors/chunjun-connector-oraclelogminer/src/main/java/com/dtstack/chunjun/connector/oraclelogminer/converter/OracleRawTypeMapper.java
@@ -75,7 +75,7 @@ public static DataType apply(TypeConfig type) {
                     if (type.getType().contains("TIME ZONE")) {
                         return DataTypes.TIMESTAMP_WITH_LOCAL_TIME_ZONE();
                     } else {
-                        return DataTypes.TIMESTAMP();
+                        return type.toTimestampDataType(0);
                     }
                 }
                 throw new UnsupportedTypeException(type);
diff --git a/chunjun-connectors/chunjun-connector-postgresql/src/main/java/com/dtstack/chunjun/connector/postgresql/dialect/PostgresqlDialect.java b/chunjun-connectors/chunjun-connector-postgresql/src/main/java/com/dtstack/chunjun/connector/postgresql/dialect/PostgresqlDialect.java
@@ -19,6 +19,7 @@
 package com.dtstack.chunjun.connector.postgresql.dialect;
 
 import com.dtstack.chunjun.config.CommonConfig;
+import com.dtstack.chunjun.connector.jdbc.config.JdbcConfig;
 import com.dtstack.chunjun.connector.jdbc.dialect.JdbcDialect;
 import com.dtstack.chunjun.connector.jdbc.statement.FieldNamedPreparedStatement;
 import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
@@ -36,6 +37,7 @@
 import java.sql.ResultSet;
 import java.util.Arrays;
 import java.util.Optional;
+import java.util.Properties;
 import java.util.stream.Collectors;
 
 public class PostgresqlDialect implements JdbcDialect {
@@ -160,4 +162,14 @@ public String getCopyStatement(
         return String.format(
                 COPY_SQL_TEMPL, tableLocation, fieldsExpression, fieldDelimiter, nullVal);
     }
+
+    @Override
+    public void putWriterExtParam(JdbcConfig jdbcConf) {
+        Properties properties = jdbcConf.getProperties();
+        if (properties == null) {
+            properties = new Properties();
+        }
+        properties.putIfAbsent("reWriteBatchedInserts", "true");
+        jdbcConf.setProperties(properties);
+    }
 }
diff --git a/chunjun-connectors/chunjun-connector-sqlserver/src/main/java/com/dtstack/chunjun/connector/sqlserver/dialect/SqlserverDialect.java b/chunjun-connectors/chunjun-connector-sqlserver/src/main/java/com/dtstack/chunjun/connector/sqlserver/dialect/SqlserverDialect.java
@@ -20,6 +20,7 @@
 
 import com.dtstack.chunjun.config.CommonConfig;
 import com.dtstack.chunjun.config.TypeConfig;
+import com.dtstack.chunjun.connector.jdbc.config.JdbcConfig;
 import com.dtstack.chunjun.connector.jdbc.dialect.JdbcDialect;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcInputSplit;
 import com.dtstack.chunjun.connector.jdbc.statement.FieldNamedPreparedStatement;
@@ -55,6 +56,7 @@
 import java.util.HashSet;
 import java.util.List;
 import java.util.Optional;
+import java.util.Properties;
 import java.util.Set;
 import java.util.function.Function;
 import java.util.stream.Collectors;
@@ -333,4 +335,16 @@ public Function<Tuple3<String, Integer, Integer>, TypeConfig> typeBuilder() {
             return typeConfig;
         });
     }
+
+    @Override
+    public void putWriterExtParam(JdbcConfig jdbcConf) {
+        Properties properties = jdbcConf.getProperties();
+        if (properties == null) {
+            properties = new Properties();
+        }
+        properties.putIfAbsent("enablePrepareOnFirstPreparedStatementCall", "true");
+        properties.putIfAbsent("disableStatementPooling", "false");
+        properties.putIfAbsent("statementPoolingCacheSize", "50");
+        jdbcConf.setProperties(properties);
+    }
 }
diff --git a/chunjun-connectors/chunjun-connector-sybase/src/main/java/com/dtstack/chunjun/connector/sybase/source/SybaseSourceFactory.java b/chunjun-connectors/chunjun-connector-sybase/src/main/java/com/dtstack/chunjun/connector/sybase/source/SybaseSourceFactory.java
@@ -20,7 +20,6 @@
 
 import com.dtstack.chunjun.config.SyncConfig;
 import com.dtstack.chunjun.connector.jdbc.source.JdbcSourceFactory;
-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;
 import com.dtstack.chunjun.connector.sybase.dialect.SybaseDialect;
 
 import org.apache.flink.streaming.api.environment.StreamExecutionEnvironment;
@@ -36,6 +35,5 @@ public SybaseSourceFactory(SyncConfig syncConfig, StreamExecutionEnvironment env
                 && jdbcConfig.getFetchSize() == 0) {
             jdbcConfig.setFetchSize(1000);
         }
-        JdbcUtil.putExtParam(jdbcConfig);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -185,6 +185,7 @@ protected JdbcConfig getSinkConnectionConfig(`
`185`	`185`
`186`	`186`	`jdbcConfig.setUniqueKey(keyFields);`
`187`	`187`	`resetTableInfo(jdbcConfig);`
	`188`	`+ getDialect().putWriterExtParam(jdbcConfig);`
`188`	`189`	`return jdbcConfig;`
`189`	`190`	`}`
`190`	`191`
`@@ -257,6 +258,7 @@ protected JdbcConfig getSourceConnectionConfig(ReadableConfig readableConfig) {`
`257`	`258`	`if (StringUtils.isBlank(jdbcConfig.getCustomSql())) {`
`258`	`259`	`resetTableInfo(jdbcConfig);`
`259`	`260`	`}`
	`261`	`+ getDialect().putReaderExtParam(jdbcConfig);`
`260`	`262`	`return jdbcConfig;`
`261`	`263`	`}`
`262`	`264`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,6 @@`
`22`	`22`	`import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormat;`
`23`	`23`	`import com.dtstack.chunjun.connector.jdbc.source.JdbcInputFormatBuilder;`
`24`	`24`	`import com.dtstack.chunjun.connector.jdbc.source.JdbcSourceFactory;`
`25`		`-import com.dtstack.chunjun.connector.jdbc.util.JdbcUtil;`
`26`	`25`	`import com.dtstack.chunjun.connector.mysql.dialect.MysqlDialect;`
`27`	`26`
`28`	`27`	`import org.apache.flink.streaming.api.environment.StreamExecutionEnvironment;`
`@@ -42,7 +41,6 @@ public MysqlSourceFactory(SyncConfig syncConfig, StreamExecutionEnvironment env)`
`42`	`41`	`&& jdbcConfig.getFetchSize() == 0) {`
`43`	`42`	`jdbcConfig.setFetchSize(1000);`
`44`	`43`	`}`
`45`		`- JdbcUtil.putExtParam(jdbcConfig);`
`46`	`44`	`}`
`47`	`45`
`48`	`46`	`@Override`