-
Notifications
You must be signed in to change notification settings - Fork 28k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-35720][SQL] Support casting of String to timestamp without time zone type #32898
Conversation
Can one of the admins verify this patch? |
|
||
test("SPARK-35720: Support casting of String to timestamp without time zone type") { | ||
specialTs.foreach { s => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I mentioned in https://issues.apache.org/jira/browse/SPARK-35720, I don't think we can simply use stringToTimestamp
here. You will find that when you have more test cases here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gengliangwang thanks for the review. Will update ASAP.
@pingsutw any update on this? The project of |
@gengliangwang Sorry, I was in the army a couple days ago. |
@pingsutw It's OK :) |
What changes were proposed in this pull request?
Extend the Cast expression and support casting of String to timestamp without time zone type
Why are the changes needed?
To conform the ANSI SQL standard which requires to support such casting.
Does this PR introduce any user-facing change?
No, the new timestamp type is not released yet.
How was this patch tested?
Unit test