pan3793 commented on PR #57831: URL: https://github.com/apache/spark/pull/57831#issuecomment-5249463008
I think the community does not currently seem to treat all error condition names as part of the public API contract, nor does it subject them to sufficiently rigorous review, but I agree we should be careful about those error conditions already used by older versions of the client for control flow. Specific to the error conditions in this PR, @nchammas, do you have a concrete suggestion for how we should handle them? Since SPARK-57933 is included in 4.3, I don't want to defer this one to the next version and accumulate the tech debt. -- 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]
