holdenk commented on pull request #29367:
URL: https://github.com/apache/spark/pull/29367#issuecomment-669665636


   > @holdenk, I am a bit confused by the commit message of the only commit in 
this PR: "Shutdown executor once we are done decommissioning". Isn't this the 
recently merged PR #29211 (it went to master right) ?
   > 
   > Can you make sure that this commit is appropriately rebased on master with 
a commit like "Use graceful decommissioning as part of dynamic scaling"
   
   Ah yeah, if you click expand you can see it's just all squished down 
together into one commit and the full commit text covers everything. When it 
gets merged the commit message is picked from the title anyways but I'll rename 
the title line of the commit.


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

Reply via email to