HeartSaVioR commented on pull request #29350: URL: https://github.com/apache/spark/pull/29350#issuecomment-669618859
@yanxiaole > If this is the idea we are going to apply, I can submit a PR about it later. Probably start from a general exception and log it? What do you think? @HeartSaVioR Yeah I think it's OK. My sketched idea is wrapping whole lambda function in filter with try-catch, and logging and swallowing `NonFatal`. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
