dongjoon-hyun opened a new pull request, #57019: URL: https://github.com/apache/spark/pull/57019
### What changes were proposed in this pull request? This PR removes the temporary file `java_opts.txt` from the K8s container `entrypoint.sh` by reading `SPARK_JAVA_OPT_*` environment variables directly into the array via process substitution, which works in both `bash` and `zsh`. ### Why are the changes needed? - `SPARK_JAVA_OPT_*` carries `spark.executor.extraJavaOptions` verbatim, which may contain sensitive values (e.g., keystore passwords). The file was written in plaintext to the working directory and never deleted, so it could outlive the process on persistent volumes. - It also fails under `readOnlyRootFilesystem: true`, even on the `driver` path that never uses the file. ### Does this PR introduce _any_ user-facing change? No, the resulting Java options are identical. ### How was this patch tested? Manually verified in both `bash` and `zsh` that options with spaces, the empty case under `set -e`, and the numeric ordering are preserved. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Fable 5 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
