sarutak opened a new pull request, #59253: URL: https://github.com/apache/spark/pull/59253
### What changes were proposed in this pull request? This backports #59187 (commit 25a9aa7) to branch-4.3. This PR fixes the "Valid values" list for `spark.kubernetes.executor.rollPolicy`, which was out of sync with the `ExecutorRollPolicy` enum in both `Config.scala` and `running-on-kubernetes.md`. - `Config.scala`: added the missing `TOTAL_SHUFFLE_WRITE` and `DISK_USED`. - `running-on-kubernetes.md`: added the missing `AVERAGE_DURATION`, `PEAK_JVM_ONHEAP_MEMORY`, `PEAK_JVM_OFFHEAP_MEMORY`, `TOTAL_SHUFFLE_WRITE`, `DISK_USED` and `OUTLIER_NO_FALLBACK`, including their per-policy descriptions. Both lists now follow the enum definition order. ### Why are the changes needed? The documented list of valid values listed only a subset of the supported policies, which is misleading. ### Does this PR introduce _any_ user-facing change? No. Docs only. No behavior change. ### How was this patch tested? Verified by inspection that the lists in both files match the branch-4.3 `ExecutorRollPolicy` enum, and that the `OUTLIER` dimensions, their checking order, and the `DISK_USED` wording match `ExecutorRollPlugin` and `AppStatusListener`. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Kiro CLI / Claude -- 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]
