sarutak opened a new pull request, #59187:
URL: https://github.com/apache/spark/pull/59187

   ### What changes were proposed in this pull request?
   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`, `ACTIVE_TASKS` 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 
`ExecutorRollPolicy` enum.
   
   ### 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]

Reply via email to