holdenk opened a new pull request #28817:
URL: https://github.com/apache/spark/pull/28817


   ### What changes were proposed in this pull request?
   
   Exit the executor when it has been asked to decommission and there is 
nothing left for it to do.
   
   ### Why are the changes needed?
   
   If we want to use decommissioning in Spark's own scale down we should 
terminate the executor once finished.
   
   
   ### Does this PR introduce _any_ user-facing change?
   
   The decommissioned executors will exit and the end of decommissioning. This 
is sort of a user facing change, however decommissioning hasn't been in any 
releases yet.
   
   ### How was this patch tested?
   
   I changed the unit test to not send the executor exit message and still wait 
on the executor exited message.


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to