From 0cdab916057b392fbd5b0ef8f1c1e97f56cdb1ec Mon Sep 17 00:00:00 2001 From: Andrey Shitov Date: Wed, 27 Nov 2024 17:48:23 +0700 Subject: [PATCH] [PATCH] Add support for handling multiple Java options in HiveProcessBuilder Updated HiveProcessBuilder.scala to properly parse and handle multiple Java options specified in ENGINE_HIVE_JAVA_OPTIONS. Instead of appending the options as a single string, they are now split and added individually to the buffer, ensuring proper handling of whitespace-separated JVM arguments. --- .../org/apache/kyuubi/engine/hive/HiveProcessBuilder.scala | 4 +--- 1 file changed, 1 insertion(+), 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 index 903e06575cc..18ce17b669b 100644 --- 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]