[GitHub] [spark] mridulm commented on pull request #40707: [SPARK-43033][SQL] Avoid task retries due to AssertNotNull checks

["mridulm (via GitHub)" <gi...@apache.org>]

mridulm commented on PR #40707:
URL: https://github.com/apache/spark/pull/40707#issuecomment-1512731238

   I would be on the conservative side and skip retry when we are absolutely certain that a retry will not help.
   SerDe failures, for example, are good candidates (which is already handled), and similar ... note that in non deterministic tasks, a retry can succeed which earlier failed with a user exception 


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org