nchammas commented on PR #57831: URL: https://github.com/apache/spark/pull/57831#issuecomment-5285251257
If we need to get something unblocked for the upcoming release, it is what it is, I guess. Let's just make it clear in the code and/or docs that: 1. This case is an exception to how things should be done. 2. We may redesign this with a compatibility layer down the line. 3. We will programmatically disallow other sub-conditions from having a different error class from their parent. (This can be implemented as a new test.) 4. We generally discourage contributors from changing sub-conditions to override the parent error state at all (even with the same error class). -- 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]
