agrawaldevesh commented on pull request #29032: URL: https://github.com/apache/spark/pull/29032#issuecomment-660432463
@holdenk ... It would be great for you to review this PR. This PR stems from your suggestion of plumbing a "isWorkerLost" to the executor decommission message, as you suggested in https://github.com/apache/spark/pull/29014#issuecomment-654420917. It does not introduce any other semantic changes. I am keeping this as a separate PR instead of folding it into https://github.com/apache/spark/pull/29014, to keep the latter easy to revert. This PR (#29032) shouldn't be introducing any semantic changes and thus should be less risky. ---------------------------------------------------------------- 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]
