Github user foxish commented on a diff in the pull request:
https://github.com/apache/spark/pull/20148#discussion_r159740606
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/steps/BasicDriverConfigurationStep.scala
---
@@ -119,7 +119,7 @@ private[spark] class BasicDriverConfigurationStep(
.endEnv()
.addNewEnv()
.withName(ENV_DRIVER_ARGS)
- .withValue(appArgs.map(arg => "\"" + arg + "\"").mkString(" "))
+ .withValue(appArgs.mkString(" "))
--- End diff --
Thanks for calling this out - let's try and test this using an integration
test as well - https://github.com/apache-spark-on-k8s/spark-integration/. Would
be good to put in manual integration testing results here until we have the PRB
automation figured out.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
