Commit 7cb737f
committed
[SPARK-52647] Fix
### What changes were proposed in this pull request?
This PR aims to fix `jvmArgs` parameter to support multiple arguments.
https://github.com/apache/spark-kubernetes-operator/blob/a50f4533225d5e2059796426cdfbdf25747dd56a/build-tools/helm/spark-kubernetes-operator/templates/spark-operator.yaml#L104-L105
### Why are the changes needed?
The previous syntax, `"$OPERATOR_JAVA_OPTS"`, cannot handle multiple Java options.
https://github.com/apache/spark-kubernetes-operator/blob/a50f4533225d5e2059796426cdfbdf25747dd56a/build-tools/docker/docker-entrypoint.sh#L29
```
$ cat test.sh
OPERATOR_JAVA_OPTS="-Dfile.encoding=UTF8 -XX:+ExitOnOutOfMemoryError"
exec java "$OPERATOR_JAVA_OPTS" -XX:+PrintFlagsFinal -version
$ sh test.sh | grep ExitOnOutOfMemoryError
bool ExitOnOutOfMemoryError = false {product} {default}
openjdk version "21.0.7" 2025-04-15
OpenJDK Runtime Environment Homebrew (build 21.0.7)
OpenJDK 64-Bit Server VM Homebrew (build 21.0.7, mixed mode, sharing)
```
### Does this PR introduce _any_ user-facing change?
Yes. This is a bug fix.
### How was this patch tested?
Manual review because JVM option check is a little tricky.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #267 from dongjoon-hyun/SPARK-52647.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>jvmArgs to support multiple arguments1 parent a50f453 commit 7cb737f
1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | | - | |
| 29 | + | |
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| |||
0 commit comments