Merge pull request #6 from arenadata/feature/hive_javaopts_fix

[PATCH] Add support for handling multiple Java options in HiveProcessBuilder
arenadata · Nov 27, 2024 · e9f01c6 · e9f01c6
2 parents 201799d + 0cdab91
commit e9f01c6
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/hive/HiveProcessBuilder.scala b/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/hive/HiveProcessBuilder.scala
@@ -62,9 +62,7 @@ class HiveProcessBuilder(
     val memory = conf.get(ENGINE_HIVE_MEMORY)
     buffer += s"-Xmx$memory"
     val javaOptions = conf.get(ENGINE_HIVE_JAVA_OPTIONS)
-    if (javaOptions.isDefined) {
-      buffer += javaOptions.get
-    }
+    javaOptions.map(_.trim.split("\\s+")).foreach(buffer ++= _)
     // -Xmx5g
     // java options
     val classpathEntries = new mutable.LinkedHashSet[String]