dongjoon-hyun opened a new pull request, #59112:
URL: https://github.com/apache/spark/pull/59112

   ### What changes were proposed in this pull request?
   
   This PR aims to prune the PVC usage reports of removed and inactive 
executors in `ExecutorPVCResizePlugin`.
   - Exclude inactive executor pods (`spark-exec-inactive=true`) from the pod 
listing, like `ExecutorPodsPollingSnapshotSource` and 
`ExecutorPodsWatchSnapshotSource`.
   - Drop the reports of executors without a listed pod, like 
`ExecutorResizePlugin` does for `cappedExecutors`.
   
   ### Why are the changes needed?
   
   `latestReports` is never pruned. So, in long-running applications with 
dynamic allocation, the reports of removed executors accumulate forever and the 
whole map is logged at INFO level every interval.
   
   In addition, with `spark.kubernetes.executor.deleteOnTermination=false`, the 
stale reports of terminated executor pods keep being applied to their PVCs, 
which can be reused by new executors.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes but previous it was never pruned. This is a bug fix for 
`ExecutorPVCResizePlugin`, which was released in 
[v4.2.0](https://github.com/apache/spark/releases/tag/v4.2.0) (2026-07-11).
   
   ### How was this patch tested?
   
   Pass the CIs with the newly added test cases.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Opus 5.5


-- 
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