-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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-36257][SQL] Updated the version of TimestampNTZ related changes as 3.3.0 #33478
Conversation
cc @beliefer as well |
Test build #141476 has finished for PR 33478 at commit
|
jenkins, retest this please |
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.
LGTM
Kubernetes integration test starting |
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.
How about:
spark/sql/core/src/main/scala/org/apache/spark/sql/functions.scala
Lines 2984 to 2986 in e9b18b0
* @since 3.2.0 | |
*/ | |
def localtimestamp(): Column = withExpr { LocalTimestamp() } |
Kubernetes integration test status success |
Kubernetes integration test unable to build dist. exiting with code: 1 |
@MaxGekk Thanks, I have updated the code. |
Kubernetes integration test unable to build dist. exiting with code: 1 |
Test build #141479 has finished for PR 33478 at commit
|
Merging to master |
Test build #141486 has finished for PR 33478 at commit
|
What changes were proposed in this pull request?
As we decided to release TimestampNTZ type in Spark 3.3, we should update the versions of TimestampNTZ related changes as 3.3.0.
Why are the changes needed?
Correct the versions in documentation/code comment.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing UT