This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git
The following commit(s) were added to refs/heads/main by this push:
new c1d8af1 [SPARK-52654] Use `ParallelGC` by default
c1d8af1 is described below
commit c1d8af1ad47b7235df36720f1bd25e0e6c9949bd
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Wed Jul 2 08:26:34 2025 -0700
[SPARK-52654] Use `ParallelGC` by default
### What changes were proposed in this pull request?
This PR aims to use `ParallelGC` by default.
### Why are the changes needed?
Since `Spark K8s Operator` is not interactive application, we had better
use `ParallelGC` because it is optimized for throughput in general.
### Does this PR introduce _any_ user-facing change?
No behavior change.
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #270 from dongjoon-hyun/SPARK-52654.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
build-tools/helm/spark-kubernetes-operator/values.yaml | 2 +-
docs/operations.md | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/build-tools/helm/spark-kubernetes-operator/values.yaml
b/build-tools/helm/spark-kubernetes-operator/values.yaml
index 92f28b4..3fb3b95 100644
--- a/build-tools/helm/spark-kubernetes-operator/values.yaml
+++ b/build-tools/helm/spark-kubernetes-operator/values.yaml
@@ -41,7 +41,7 @@ operatorDeployment:
#
https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/
topologySpreadConstraints: [ ]
operatorContainer:
- jvmArgs: "-Dfile.encoding=UTF8 -XX:+ExitOnOutOfMemoryError"
+ jvmArgs: "-Dfile.encoding=UTF8 -XX:+ExitOnOutOfMemoryError
-XX:+UseParallelGC"
env:
- name: "SPARK_USER"
value: "spark"
diff --git a/docs/operations.md b/docs/operations.md
index 96a58b6..4f679a8 100644
--- a/docs/operations.md
+++ b/docs/operations.md
@@ -76,7 +76,7 @@ following table:
| operatorDeployment.operatorPod.volumes |
Additional volumes to be added to the operator pod.
|
|
| operatorDeployment.operatorPod.priorityClassName | Priority
class name to be used for the operator pod
|
|
| operatorDeployment.operatorPod.securityContext | Security
context overrides for the operator pod
|
|
-| operatorDeployment.operatorPod.operatorContainer.jvmArgs | JVM arg
override for the operator container.
| `"-Dfile.encoding=UTF8 -XX:+ExitOnOutOfMemoryError"`
|
+| operatorDeployment.operatorPod.operatorContainer.jvmArgs | JVM arg
override for the operator container.
| `"-Dfile.encoding=UTF8 -XX:+ExitOnOutOfMemoryError
-XX:+UseParallelGC"` |
| operatorDeployment.operatorPod.operatorContainer.env | Custom
env to be added to the operator container.
|
|
| operatorDeployment.operatorPod.operatorContainer.envFrom | Custom
envFrom to be added to the operator container, e.g. for downward API.
|
|
| operatorDeployment.operatorPod.operatorContainer.probes | Probe
config for the operator container.
|
|
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
