Skip to content

fix(athena): change PyAthena to >=3.9.0 for timestamp with timezone #2714

Open
bujjibabukatta wants to merge 6 commits into
sodadata:mainfrom
bujjibabukatta:fix/2402-pyathena
Open

fix(athena): change PyAthena to >=3.9.0 for timestamp with timezone #2714
bujjibabukatta wants to merge 6 commits into
sodadata:mainfrom
bujjibabukatta:fix/2402-pyathena

Conversation

@bujjibabukatta
Copy link
Copy Markdown

Fixes #2402

Changes

  • Bumped PyAthena dependency from <3.0,>=2.2.0 to >=3.9.0
  • Removed TIMESTAMP_TZTIMESTAMP synonym (was silently
    dropping timezone info)
  • Mapped TIMESTAMP_TZ"timestamp with time zone" in type
    mappings
  • Refactored format_metadata_data_type to handle types like
    timestamp(3) with time zone correctly using regex

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


bujjibabukatta seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@bujjibabukatta bujjibabukatta changed the title fix(athena): change PyAthena to >=3.9.0 for timestamp with timezone s… fix(athena): change PyAthena to >=3.9.0 for timestamp with timezone May 18, 2026
@bujjibabukatta
Copy link
Copy Markdown
Author

Hi @milanaleksic could you please verify and approve pull request?

@bujjibabukatta
Copy link
Copy Markdown
Author

Hi @milanaleksic all qc check passed ,could you verify and approve pull request.

@bujjibabukatta
Copy link
Copy Markdown
Author

Hi @m1n0 could you please approve pull request?

@sonarqubecloud
Copy link
Copy Markdown

@bujjibabukatta
Copy link
Copy Markdown
Author

bujjibabukatta commented May 27, 2026

Hi @Niels-b @m1n0 @paulteehan @AidaPaul @mivds @nik843 @Lepo00 @milanaleksic @nielsn @jzalucki all tests passed, could you please review and approve pull request ?

@AidaPaul
Copy link
Copy Markdown
Contributor

@bujjibabukatta after a glance I have a lot of questions as this seems to introduce a breaking changes, downgrade soda code, bump PyAthena by major version etc...

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.

Support for datetime with timezone in soda-core-athena with updated pyathena >= 3.9.0

3 participants