Skip to content

Trino: support config timezone - #35147

Closed
duyet wants to merge 2 commits into
apache:mainfrom
duyet:main
Closed

duyet wants to merge 2 commits into
apache:mainfrom
duyet:main

Conversation

@duyet

@duyet duyet commented Oct 24, 2023

Copy link
Copy Markdown
Contributor

Support config trino timezone by providing it in connection extra field

{ "timezone": "Asia/Ho_Chi_Minh" }

https://github.com/trinodb/trino-python-client?tab=readme-ov-file#timezone


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Comment thread airflow/providers/trino/hooks/trino.py Resolved
@duyet
duyet force-pushed the main branch 2 times, most recently from 7754a5c to 0569202 Compare October 24, 2023 08:10
@Taragolis

Copy link
Copy Markdown
Contributor

And also tests should be adopted to the changes

@eladkal

eladkal commented Nov 2, 2023

Copy link
Copy Markdown
Contributor

@duyet can you please add a unit test?

@eladkal

eladkal commented Dec 1, 2023

Copy link
Copy Markdown
Contributor

Closed in favor of #35963

@eladkal eladkal closed this Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants