Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/19738
> Yup, it looks so. Could we add another small test case for it as well?
There is already, it's just that `str()` will convert from unicode as long
as it's only ASCII characters so it passes. Do you think we should use non
ASCII in tests? I'll check to see if that's done elsewhere in Spark..
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
