xianjingfeng commented on PR #85: URL: https://github.com/apache/incubator-uniffle/pull/85#issuecomment-1196192687
> I can’t got your thought fully. As i know, when need to make one shuffle server decommission, just add it to the exclude file in coordinator side. There is no need to restart coordinator/shuffle servers. > > This PR is just to make shuffle server process exit once all apps finished. Sorry, i don't know `excludeNodes` update periodically before. But if we have multi coordinator, we will change exclude-node-files multi times, and if we need to start shuffle server again, we need change them again. And i think _**exclude**_ is not equal to _**decommission**_ -- 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]
