HeartSaVioR commented on pull request #33586: URL: https://github.com/apache/spark/pull/33586#issuecomment-891626167
Thanks for the updates! The change looks OK, but I'm a bit cautious that we lose ability to indicate such case. What about logging via INFO, or DEBUG if you feel this would be too verbose? The case might be red-herring so I agree IllegalArgumentException is odd, but sometimes the case could represent the real problem and we have no log on skipping it if I understand correctly. It'd be nice if we leave some information on debugging. -- 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]
