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 f9b4242 [SPARK-55738] Fix `ResourceRetainPolicy` javadoc location
f9b4242 is described below
commit f9b424201520e5900c8faf6021e8ffbe802a750d
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Feb 26 20:22:30 2026 -0800
[SPARK-55738] Fix `ResourceRetainPolicy` javadoc location
### What changes were proposed in this pull request?
This PR aims to fix `ResourceRetainPolicy` java location by moving the
comment.
### Why are the changes needed?
The current location is misleading and wrong.
<img width="534" height="475" alt="Screenshot 2026-02-26 at 20 08 22"
src="https://github.com/user-attachments/assets/91368fae-a945-4f5b-915a-6f4adf47240d";
/>
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: `Gemini 3.1 Pro (High)` on `Antigravity`
Closes #529 from dongjoon-hyun/SPARK-55738.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.../k8s/operator/spec/ResourceRetainPolicy.java | 26 +++++++++++-----------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git
a/spark-operator-api/src/main/java/org/apache/spark/k8s/operator/spec/ResourceRetainPolicy.java
b/spark-operator-api/src/main/java/org/apache/spark/k8s/operator/spec/ResourceRetainPolicy.java
index 322d82f..4cce4c8 100644
---
a/spark-operator-api/src/main/java/org/apache/spark/k8s/operator/spec/ResourceRetainPolicy.java
+++
b/spark-operator-api/src/main/java/org/apache/spark/k8s/operator/spec/ResourceRetainPolicy.java
@@ -19,20 +19,20 @@
package org.apache.spark.k8s.operator.spec;
+/**
+ * Configure operator to delete / retain resources for an app after it
terminates. This controls
+ * the resources created directly by Operator (spark-submit), including driver
pod and its
+ * configmap. Resources created by driver are still controlled by driver (via
SparkConf). Tuning
+ * 'spark.kubernetes.driver.service.deleteOnTermination' and
`spark.kubernetes.executor
+ * .deleteOnTermination`in application for resource retaining policy is still
necessary. Default
+ * to 'Never' so operator deletes all created Spark resources after app
terminates. Setting this
+ * to 'Always' / 'OnFailure' to indicate operator not to delete resources
based on app state, this
+ * can be helpful in dev phase to debug failed Spark pod behavior. Driver has
owner reference
+ * configured to make sure it's garbage collected by k8s. Please be advised
that resource would
+ * not be retained if application is configured to restart, this is to avoid
resource quota usage
+ * increase unexpectedly or resource conflicts among multiple attempts.
+ */
public enum ResourceRetainPolicy {
- /**
- * Configure operator to delete / retain resources for an app after it
terminates. This controls
- * the resources created directly by Operator (spark-submit), including
driver pod and its
- * configmap. Resources created by driver are still controlled by driver
(via SparkConf). Tuning
- * 'spark.kubernetes.driver.service.deleteOnTermination' and
`spark.kubernetes.executor
- * .deleteOnTermination`in application for resource retaining policy is
still necessary. Default
- * to 'Never' so operator deletes all created Spark resources after app
terminates. Setting this
- * to 'Always' / 'OnFailure' to indicate operator not to delete resources
based on app state, this
- * can be helpful in dev phase to debug failed Spark pod behavior. Driver
has owner reference
- * configured to make sure it's garbage collected by k8s. Please be advised
that resource would
- * not be retained if application is configured to restart, this is to avoid
resource quota usage
- * increase unexpectedly or resource conflicts among multiple attempts.
- */
Always,
OnFailure,
Never
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
