Skip to content

Commit eefb1fa

Browse files
authored
[Fix-17073]Fix DataSource/JDBC connection failure (#17081)
1 parent 19cb333 commit eefb1fa

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/main/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public String getJdbcUrl(ConnectionParam connectionParam) {
112112
String jdbcUrl = kyuubiConnectionParam.getJdbcUrl();
113113

114114
if (MapUtils.isNotEmpty(kyuubiConnectionParam.getOther())) {
115-
return jdbcUrl + "?" + transformOther(kyuubiConnectionParam.getOther());
115+
return jdbcUrl + ";" + transformOther(kyuubiConnectionParam.getOther());
116116
}
117117
return jdbcUrl;
118118
}

dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/test/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ public void testGetJdbcUrl() {
136136
Map<String, String> other = new HashMap<>();
137137
other.put("serverTimezone", "Asia/Shanghai");
138138
connectionParam.setOther(other);
139-
Assertions.assertEquals("jdbc:kyuubi://localhost1:5142,localhost2:5142/default?serverTimezone=Asia/Shanghai",
139+
Assertions.assertEquals("jdbc:kyuubi://localhost1:5142,localhost2:5142/default;serverTimezone=Asia/Shanghai",
140140
kyuubiDatasourceProcessor.getJdbcUrl(connectionParam));
141141
}
142142

0 commit comments

Comments
 (0)