Skip to content
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

[auto-merge] branch-23.12 to branch-24.02 [skip ci] [bot] #9882

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

nvauto
Copy link
Collaborator

@nvauto nvauto commented Nov 28, 2023

auto-merge triggered by github actions on branch-23.12 to create a PR keeping branch-24.02 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

…#9810)

* Updates to test suite to start using GpuTimeZoneDB

Signed-off-by: Navin Kumar <[email protected]>

* Initialize the GPU timezone database in the plugin initialization for executor, use this in tests

Signed-off-by: Navin Kumar <[email protected]>

* Remove GpuTimeZoneDB init from plugin for now, eventually add again and hide behind a config flag when first expression using it is completed.

Signed-off-by: Navin Kumar <[email protected]>

* Add plugin initialization code and change config flag for non UTC timezone support

Signed-off-by: Navin Kumar <[email protected]>

* Use GpuTimeZoneDB to run from_utc_timestamp on the GPU for non-UTC non-DST timezones

Signed-off-by: Navin Kumar <[email protected]>

* Fix formatting of includes here

Signed-off-by: Navin Kumar <[email protected]>

* Add some Olson timezones to fallback test and add config option to test proper fallback

Signed-off-by: Navin Kumar <[email protected]>

* Add fallback test for when config option is not enabled

Signed-off-by: Navin Kumar <[email protected]>

---------

Signed-off-by: Navin Kumar <[email protected]>
@nvauto nvauto merged commit cd4e832 into branch-24.02 Nov 28, 2023
@nvauto
Copy link
Collaborator Author

nvauto commented Nov 28, 2023

SUCCESS - auto-merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants