You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/05/13 07:01:56 UTC
[GitHub] [spark] cloud-fan commented on issue #24567: [SPARK-27638][SQL]:
Cast string to date/timestamp in binary comparisons with dates/timestamps
cloud-fan commented on issue #24567: [SPARK-27638][SQL]: Cast string to date/timestamp in binary comparisons with dates/timestamps
URL: https://github.com/apache/spark/pull/24567#issuecomment-491700227
A general note: eventually we do need to support different type coercion modes for different users. But the type coercion rule should be reasonable under any mode. It's not reasonable to compare date/timestamp and string by casting to string, as it returns the wrong result when the string is a partial date/timestamp.
For safety, we can add a legacy config to fallback to the old behavior.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org