Yicong-Huang commented on PR #55670: URL: https://github.com/apache/spark/pull/55670#issuecomment-4375377751
> Well, this is not about breaking backward compatibility, @Yicong-Huang . Instead, we are talking about more forward compatibility which we didn't promise at all for the error messages. Yes! I totally understand. The intention to fix those error messages is actually aligned: I believe the previous version does not have great future compatibility considered. For this specific case, the error class can only be thrown from a specific eval type, then in the future if we need to add more eval types (which we did), the new eval types will have to create a new error class instead of reusing the same. I think the effort is to make it more future compatible. -- 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]
