[AMORO-2862] fix bug: A null pointer error occurs when using S3 as St…

…orage when using the kyuubi terminal. (#2863) * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. * [AMORO-2862] Fix bug #2862 A null pointer error occurs when using S3 as Storage when using the kyuubi terminal. --------- Co-authored-by: zhaoshuaihua <zhaoshuaihua@kingsoft.com> Co-authored-by: ZhouJinsong <zhoujinsong0505@163.com> Co-authored-by: baiyangtx <xiangnebula@163.com>
apache · May 30, 2024 · ad46f97 · ad46f97
1 parent de0e6ff
commit ad46f97
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...r/src/main/java/org/apache/amoro/server/terminal/kyuubi/KyuubiTerminalSessionFactory.java b/...r/src/main/java/org/apache/amoro/server/terminal/kyuubi/KyuubiTerminalSessionFactory.java
@@ -37,6 +37,7 @@
 import java.sql.SQLException;
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.Properties;
 import java.util.stream.Collectors;
 
@@ -123,7 +124,7 @@ public TerminalSession create(TableMetaStore metaStore, Configurations configura
     sessionConf.put("jdbc.url", kyuubiJdbcUrl);
     Properties properties = new Properties();
 
-    if (!metaStore.isKerberosAuthMethod()) {
+    if (!metaStore.isKerberosAuthMethod() && Objects.nonNull(metaStore.getHadoopUsername())) {
       properties.put(JdbcConnectionParams.AUTH_USER, metaStore.getHadoopUsername());
       sessionConf.put(JdbcConnectionParams.AUTH_USER, metaStore.getHadoopUsername());
     }