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

Pin Java version as 23.0.1 #24786

Closed
wants to merge 1 commit into from
Closed

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Jan 24, 2025

Description

23.0.2 upgraded IANA Time Zone Database to 2024b.
It changed a gap in America/Bahia_Banderas.
openjdk/jdk23u@73b2341

We changed the timezone from Asia/Katmandu in prestodb/presto#10128.

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@martint
Copy link
Member

martint commented Jan 24, 2025

I don’t think that’s the right approach. We should fix the tests, instead.

@ebyhr
Copy link
Member Author

ebyhr commented Jan 24, 2025

Right, we should find a new timezone. This is a workaround until finding such timezone.

@martint
Copy link
Member

martint commented Jan 24, 2025

Which tests are failing? We should also consider writing tests that are not sensitive to specific timezone rules. They used to be needed for the old timestamp semantics, but I don't think that's the case anymore.

@ebyhr
Copy link
Member Author

ebyhr commented Jan 24, 2025

23 jobs (mostly type mapping tests in connectors) are failing: https://github.com/trinodb/trino/actions/runs/12941902891/job/36098727148

23.0.2 upgraded IANA Time Zone Database to 2024b.
It changed a gap in America/Bahia_Banderas.
@wendigo
Copy link
Contributor

wendigo commented Jan 24, 2025

@ebyhr no such timezone exists anymore that has gap in 1970-01-01

@wendigo wendigo mentioned this pull request Jan 24, 2025
@ebyhr ebyhr closed this Jan 25, 2025
@ebyhr ebyhr deleted the ebi/java-23.0.1 branch January 25, 2025 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants