gaogaotiantian commented on PR #53099: URL: https://github.com/apache/spark/pull/53099#issuecomment-3613931626
Okay this is finally ready to be reviewed. Some tests were fixed - they were checking the end `\n` in the exception message but the exception string itself should not contain an `\n` otherwise there will be an blank line after when being printed as stack trace. Also it's regex check so I don't think we need the `\n`. Some of the exception types were changed slightly - it's breaking change in theory but should have minor impact. -- 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]
