MaxGekk commented on PR #40632: URL: https://github.com/apache/spark/pull/40632#issuecomment-1586087359
> will make sure if some error class have subclass, then can't use parent class directly. I see. How about to add a default sub-class like `WITHOUT_SUGGESTION` or `DEFAULT` as we do in other classes? And trigger it from `malformedRecordsDetectedInRecordParsingError()`. Don't think that changing the existing approach in this PR is good idea. Let's discuss this separately (according to your approach, we should remove all sub-classes `WITHOUT_SUGGESTION`). -- 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]
