pan3793 commented on PR #52641: URL: https://github.com/apache/spark/pull/52641#issuecomment-3420387128
@HyukjinKwon Having another function looks too overkill to me. @beliefer @dongjoon-hyun @HyukjinKwon I tested two additional popular RDBMS, MariaDB and PostgreSQL, both support `version()` function, and the results contain vender suffix. If there are concerns about breaking changes for vendor Spark distributions, how about introducing an internal config, e.g., `spark.sql.legacy.useShortVersion.enabled`? But given this does not change behavior for Apache releases, I think we don't need to expose it to docs and migration guide. -- 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]
