(dolphinscheduler) branch dev updated: [Fix-17073]Fix DataSource/JDBC connection failure (#17081)

[zihaoxiang]

This is an automated email from the ASF dual-hosted git repository.

zihaoxiang pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git


The following commit(s) were added to refs/heads/dev by this push:
     new eefb1fafd2 [Fix-17073]Fix DataSource/JDBC connection failure (#17081)
eefb1fafd2 is described below

commit eefb1fafd2b79e733c82d6d739799023ac6bf2e1
Author: Aryan Kumar <154001177+tusar...@users.noreply.github.com>
AuthorDate: Thu Mar 27 12:38:22 2025 +0530

    [Fix-17073]Fix DataSource/JDBC connection failure (#17081)
---
 .../plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java       | 2 +-
 .../plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/main/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java
 
b/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/main/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java
index 72129b87ac..b4565804df 100644
--- 
a/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/main/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java
+++ 
b/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/main/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessor.java
@@ -112,7 +112,7 @@ public class KyuubiDataSourceProcessor extends 
AbstractDataSourceProcessor {
         String jdbcUrl = kyuubiConnectionParam.getJdbcUrl();
 
         if (MapUtils.isNotEmpty(kyuubiConnectionParam.getOther())) {
-            return jdbcUrl + "?" + 
transformOther(kyuubiConnectionParam.getOther());
+            return jdbcUrl + ";" + 
transformOther(kyuubiConnectionParam.getOther());
         }
         return jdbcUrl;
     }
diff --git 
a/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/test/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java
 
b/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/test/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java
index a18ceb4216..fc063371f8 100644
--- 
a/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/test/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java
+++ 
b/dolphinscheduler-datasource-plugin/dolphinscheduler-datasource-kyuubi/src/test/java/org/apache/dolphinscheduler/plugin/datasource/kyuubi/param/KyuubiDataSourceProcessorTest.java
@@ -136,7 +136,7 @@ public class KyuubiDataSourceProcessorTest {
         Map<String, String> other = new HashMap<>();
         other.put("serverTimezone", "Asia/Shanghai");
         connectionParam.setOther(other);
-        
Assertions.assertEquals("jdbc:kyuubi://localhost1:5142,localhost2:5142/default?serverTimezone=Asia/Shanghai",
+        
Assertions.assertEquals("jdbc:kyuubi://localhost1:5142,localhost2:5142/default;serverTimezone=Asia/Shanghai",
                 kyuubiDatasourceProcessor.getJdbcUrl(connectionParam));
     }