agrawaldevesh opened a new pull request #29032: URL: https://github.com/apache/spark/pull/29032
### What changes were proposed in this pull request? This PR is a giant plumbing PR that plumbs a DecommissionInfo along with the DecommissionExecutor message. ### Why are the changes needed? The motivation is to know whether a decommissioned executor would also be loosing shuffle files -- and thus it is important to know whether the worker (node) would also be decommissioned. This is similar to the existing `workerLost` field in the `SlaveLost` message. We can also embellish DecommissionInfo in the future for knowing how long the executor has (for spot kills) and the like. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Tweaked an existing unit test in `AppClientSuite` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
