Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22414
Yeah, the test that failed here asserts that it's an `AnalysisException`. I
guess it could be removed. The thing is, many other cases are still handled as
`AnalysisException`. Maybe it's best to stay consistent; I didn't realize this.
Is there any other advantage? seems like it fails just as fast either way?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
