cloud-fan commented on PR #49879: URL: https://github.com/apache/spark/pull/49879#issuecomment-2650573926
We used to treat string arguments as column names in the function APIs, and we found it was a bad idea later on, as it can be ambiguous with string literal parameters. For now, I think all function arguments should be `Column`, with additional native type arguments (string, int, etc.) for literal values. That being said, I'm also +1 to keep the `Column` version for API consistency. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
