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

Reply via email to