Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19249
Thanks for double checking @ueshin.
Yes, I noticed that too while reviewing it. I just decided to merge it as
is because I am quite sure of this one given struct type is the root type and
this case looks quite common, and regarding that it looks the first
contribution. Even though this one has a downside, practically the improvement
looked better.
I am also fine with doing this for others too (I am +0 for other types).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
