rsc: Don't set spark configuration values to an empty string

It's better to leave out a key rather than set the value to an empty string
apache · Aug 22, 2023 · 222b0d9 · 222b0d9
1 parent 074d6a2
commit 222b0d9
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/rsc/src/main/java/org/apache/livy/rsc/ContextLauncher.java b/rsc/src/main/java/org/apache/livy/rsc/ContextLauncher.java
@@ -254,7 +254,9 @@ public void run() {
 
   private static void merge(RSCConf conf, String key, String livyConf, String sep) {
     String confValue = Utils.join(Arrays.asList(livyConf, conf.get(key)), sep);
-    conf.set(key, confValue);
+    if (confValue != null && !confValue.isEmpty()) {
+      conf.set(key, confValue);
+    }
   }
 
   /**