You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "revans2 (via GitHub)" <gi...@apache.org> on 2023/03/24 21:36:58 UTC
[GitHub] [spark] revans2 commented on pull request #40524: [SPARK-42898][SQL] Mark that string/date casts do not need time zone id
revans2 commented on PR #40524:
URL: https://github.com/apache/spark/pull/40524#issuecomment-1483436510
@MaxGekk This might take a little while. I don't want to delete the test, but I also cannot just switch the data types over to Timestamp because the partition string is not always stored in a format that is compatible with TimestampType. So I ma going to have to spend some more time understanding the test to see if there is a good way to update it for TimestampType.
--
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-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org