Commit ddefa44
committed
[SPARK-52667] Improve
### What changes were proposed in this pull request?
This PR aims to improve `SparkAppReconciler` to show app names during cleaning up.
### Why are the changes needed?
**BEFORE**
The log message doesn't provide any specific information.
```
25/07/03 05:54:22 INFO SparkAppReconciler: Cleaning up resources for SparkApp.
25/07/03 05:54:22 INFO SparkAppReconciler: Cleanup completed
25/07/03 05:54:22 INFO SparkAppReconciler: Cleaning up resources for SparkApp.
25/07/03 05:54:22 INFO SparkAppReconciler: Cleanup completed
```
**AFTER**
It provides a concise and specific information.
```
25/07/03 06:06:57 INFO SparkAppReconciler: Cleaning up resources for SparkApp:pi
25/07/03 06:07:02 INFO SparkAppReconciler: Cleaning up resources for SparkApp:pi-java21
```
### Does this PR introduce _any_ user-facing change?
No behavior change. This is only changing log message information.
### How was this patch tested?
Manual build, install, test, and check logs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #275 from dongjoon-hyun/SPARK-52667.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>SparkAppReconciler to show app names during cleaning up1 parent ad6ba0d commit ddefa44
File tree
1 file changed
+2
-2
lines changed- spark-operator/src/main/java/org/apache/spark/k8s/operator/reconciler
1 file changed
+2
-2
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
186 | | - | |
| 186 | + | |
187 | 187 | | |
188 | 188 | | |
189 | 189 | | |
| |||
200 | 200 | | |
201 | 201 | | |
202 | 202 | | |
203 | | - | |
| 203 | + | |
204 | 204 | | |
205 | 205 | | |
206 | 206 | | |
| |||
0 commit comments