Github user jose-torres commented on the issue: https://github.com/apache/spark/pull/20413 +1. I originally wrote this line, and I'm reasonably confident that (as indicated by the comment) I didn't intend to check anything other than the nullity of lastExecution. I continue to be confused about why I wrote such a strange null check.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org