Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20186
> I think it's fine. We usually don't test if a config can be correctly set
as it's so trivial...
The point is to test that the code using that config works as expected, not
that it can be set. And as this PR proves, code can be wrong even in "trivial"
cases.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
