Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/19142 I'm -1 on this PR. This PR actually fix nothing instead of adding one trace log, also usually user will not enable trace log, so this one line fix is not so helpful. You can find many place in Spark where it has no "else" branch, are you going to fix them all?
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org