Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20021
Honestly, I liked very much doing the test only for testing and not
throwing an exception in production. IMHO it is an overkill to throw an
exception in production and in the remote case that we happen to forget one
place where this check can throw the exception, but it is not an issue, as it
is perfectly possible, this would also cause a regression.
Thus, honestly I am strongly against this solution.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
