Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20637
> If someone would unintentionally generate incorrect dataType, we must
detect the mistake by failing the test without an exception.
This is the part I don't agree with. If someone writes a bad UT, it is good
to get an exception. It is also an hint that the problem is in the UT itself,
rather than in the code. But we already discussed this and we have different
opinion on this. :)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
