-
Notifications
You must be signed in to change notification settings - Fork 230
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
[FEA] Support time zones that are not UTC #6839
Labels
epic
Issue that encompasses a significant feature or body of work
feature request
New feature or request
Comments
revans2
added
feature request
New feature or request
? - Needs Triage
Need team to review and classify
epic
Issue that encompasses a significant feature or body of work
labels
Oct 18, 2022
Not sure if |
Follow on issue: #9570 |
Not planing work on this for release 24.04 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
epic
Issue that encompasses a significant feature or body of work
feature request
New feature or request
Is your feature request related to a problem? Please describe.
This is related to #6603 but a super set of it. I will list out the issues here to complete both sets of work, and then reference some of them in #6603 too.
The following are the tasks to do:
Phase I (P0) - Release 23.12
current_date
expression function with CST (UTC + 8) timezone support #9805Phase II (P0) - Release 24.02
unix_timestamp
for non-DST timezones #9815unix_timestamp
for non-UTC time zones #10079spark.rapids.sql.nonUTC.enabled
configuration option #9881Phase III/Uncategorized (P1)
ParseToTimestamp
for non-UTC time zones #10006test_cast_string_ts_valid_format
failed atseed = 1701362564
#9916Uncategorized
TIMESTAMP_LTZ
type in integration testjson_ts_formats_round_trip_ntz
case #9912csv_infer_schema_timestamp_ntz
case #9913Test improvements
Benchmarking
In general we are going to try to have timestamps follow what Spark does and store them in UTC. Then convert the inputs to the desired timezone whenever we need to operate on it in a different time zone.
We should also go through other operators and see if there are things that we can/should support once we can support timezones.
The text was updated successfully, but these errors were encountered: