Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/21342
  
    Basically, I agree with this two-step direction. However, I think that it 
is too risky to simply throw and catch `SparkException`. This is because other 
`SparkException` may be caught in step 2. For example, [this 
line](https://github.com/apache/spark/pull/21342/files#diff-b024f971b46c65ddb0e4f9dcd3a831f7L100)
 throws `SparkException`.
    
    Can we throw and catch another Exception or filter the caught 
`SparkException` with certain string?


---

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

Reply via email to