You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2025/05/12 04:57:49 UTC
(kyuubi) branch branch-1.10 updated: [KYUUBI #7051] Fix usage without sslTrustStore of JDBC driver
This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch branch-1.10
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/branch-1.10 by this push:
new 8cb81873b4 [KYUUBI #7051] Fix usage without sslTrustStore of JDBC driver
8cb81873b4 is described below
commit 8cb81873b4190a35966a31387944a37a7b879145
Author: Igor Khrol <kh...@gmail.com>
AuthorDate: Mon May 12 12:57:27 2025 +0800
[KYUUBI #7051] Fix usage without sslTrustStore of JDBC driver
### Why are the changes needed?
If `sslTrustStore` is not provided `org.apache.hadoop.conf.Configuration` class existence becomes a hard dependency.
This makes jdbc client too complex to configure: extra Hadoop jars should be provided.
`hadoopCredentialProviderAvailable` variable is useless in the previous implementation logic because it's always `true` or the code is not reachable.
<img width="898" alt="Screenshot 2025-05-09 at 13 05 12" src="https://github.com/user-attachments/assets/6d202555-38c6-40d2-accb-eb78a3d4184e" />
### How was this patch tested?
Build jar and used it to connect from DataGrip.
<img width="595" alt="Screenshot 2025-05-09 at 13 01 29" src="https://github.com/user-attachments/assets/c6e4d904-a3dd-4d3f-9bdd-8bb47ed1e834" />
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #7058 from Khrol/master.
Closes #7051
b594757a0 [Igor Khrol] JDBC driver: allow usage without sslTrustStore
Authored-by: Igor Khrol <kh...@gmail.com>
Signed-off-by: Cheng Pan <ch...@apache.org>
(cherry picked from commit 61487acfa022ad9abfc538ed8b63fd02acc3a628)
Signed-off-by: Cheng Pan <ch...@apache.org>
---
kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/Utils.java | 1 -
1 file changed, 1 deletion(-)
diff --git a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/Utils.java b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/Utils.java
index 86b397437e..39e5275a5c 100644
--- a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/Utils.java
+++ b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/Utils.java
@@ -669,7 +669,6 @@ public class Utils {
hadoopCredentialProviderAvailable = true;
} catch (Exception exception) {
LOG.warn("Hadoop credential provider is unavailable", exception);
- throw new RuntimeException(exception);
}
}
if (password == null && hadoopCredentialProviderAvailable) {