Commit e5e751b
committed
[SPARK-48887][K8S] Enable
### What changes were proposed in this pull request?
This PR aims to enable `spark.kubernetes.executor.checkAllContainers` by default from Apache Spark 4.0.0.
### Why are the changes needed?
Since Apache Spark 3.1.0, `spark.kubernetes.executor.checkAllContainers` is supported and useful because [sidecar pattern](https://kubernetes.io/docs/concepts/workloads/pods/sidecar-containers/) is used in many cases. Also, it prevents user mistakes which forget and ignore the sidecars' failures by always reporting sidecar failures via executor status.
- #29924
### Does this PR introduce _any_ user-facing change?
- This configuration is no-op when there is no other container.
- This will report user containers' error correctly when there exist other containers which are provided by the users.
### How was this patch tested?
Both `true` and `false` are covered by our CI test coverage since Apache Spark 3.1.0.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #47337 from dongjoon-hyun/SPARK-48887.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>spark.kubernetes.executor.checkAllContainers by default1 parent 115c6e4 commit e5e751b
File tree
3 files changed
+4
-2
lines changed- docs
- resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s
3 files changed
+4
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| 41 | + | |
| 42 | + | |
41 | 43 | | |
42 | 44 | | |
43 | 45 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1327 | 1327 | | |
1328 | 1328 | | |
1329 | 1329 | | |
1330 | | - | |
| 1330 | + | |
1331 | 1331 | | |
1332 | 1332 | | |
1333 | 1333 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
711 | 711 | | |
712 | 712 | | |
713 | 713 | | |
714 | | - | |
| 714 | + | |
715 | 715 | | |
716 | 716 | | |
717 | 717 | | |
| |||
0 commit comments