Github user eyalfa commented on the issue:
https://github.com/apache/spark/pull/14444
@HyukjinKwon, thank you very much for your analysis.
if you read the history of this PR you'd see that at some point @hvanhovell
suggested that we completely remove CreateStruct and CreateStructUnsafe and
just leave a constructor that create the named version.
I've modified catalyst tests that relied on CreateStruct, so I guess R
tests should be modified as well.
One thing I don't really understand, which is probably related to my
(complete) lack of R knowledge:
in scala API collect returns rows, what does it return in R, what does the
'named_struct(...)' come from? is it the column name in the schema?
@hvanhovell: how strong is the contract of assigning a name to an unnamed
column? should we alias the constructed tree with the backward compatible name?
(when creating the named struct)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
