cloud-fan commented on pull request #29788: URL: https://github.com/apache/spark/pull/29788#issuecomment-695959615
This PRs adds 5 classes to represent 5 different decommission reasons, but we don't really have 5 different branches to handle these reasons. I think the abstraction should be based on the real requirements, can we simplify them? ---------------------------------------------------------------- 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]
