zero323 commented on pull request #33428: URL: https://github.com/apache/spark/pull/33428#issuecomment-883289790
I am sitting on the fence here. While ```python Row("Alice", 11) ``` is a technically valid initialization of a `Row`, I always considered it actual usage a code-smell. Hence the choice of annotations, where `*args` form is intended to be used with combination with `__call__`. Using it to initialize unnamed `Rows` is not only rather inefficient, but also leaves us with this awkward object, which is intended as factory of named `Rows`, but never used as such. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org